Skip to content

Rename remaining index files #1211

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
Tracked by #933
presleyp opened this issue Apr 28, 2022 · 2 comments · Fixed by #1397
Closed
Tracked by #933

Rename remaining index files #1211

presleyp opened this issue Apr 28, 2022 · 2 comments · Fixed by #1397
Assignees
@Kira-Pilot
Labels
site Area: frontend dashboard
Milestone
V2 Beta

Comments

@presleyp
Copy link
Contributor

presleyp commented Apr 28, 2022 •
edited
Loading

We decided not to have files named index in the frontend (site/src), and have renamed most of them, but there are a few stragglers like api/index.ts and testHelpers/index.ts. Rename these to something descriptive.
@presleyp presleyp added site Area: frontend dashboard needs grooming 🪒 labels Apr 28, 2022
@misskniss misskniss added this to the V2 Beta milestone May 3, 2022
@misskniss misskniss mentioned this issue May 6, 2022
Closed
9 tasks
@greyscaled
Copy link
Contributor

@Kira-Pilot This is some low-hanging fruit to get used to our conventions - do you think you have capacity to take it on sometime this week?

@Kira-Pilot
Copy link
Member

@vapurrmaid sure do!

@Kira-Pilot Kira-Pilot mentioned this issue May 11, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Kira-Pilot Kira-Pilot

Labels
site Area: frontend dashboard
Projects
None yet
Milestone
V2 Beta
Development

Successfully merging a pull request may close this issue.

chore: renaming index files coder/coder
4 participants
@presleyp @misskniss @greyscaled @Kira-Pilot