Skip to content
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.

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

[docs] detail how eslinting/commitlinting works #33

Open
J-Vaughan opened this issue Nov 1, 2020 · 1 comment · May be fixed by #26
Open

[docs] detail how eslinting/commitlinting works #33

J-Vaughan opened this issue Nov 1, 2020 · 1 comment · May be fixed by #26
Assignees
Labels
Effort: 1 pts 1/2-3 hours Scope: Docs Status: To Do Can and should be resolved

Comments

@J-Vaughan
Copy link
Contributor

We should summarize the airbnb guidelines, with our modifications, and having a list of the available scopes/tasks available from commitlint would be very helpful as well.

Also, issues. So far I've seen a mix of commitlint (with scope in []) and just human titles. I don't know, I guess it doesn't really matter too much what format our issues follow, because there's no real record of them in the repository.

@J-Vaughan J-Vaughan added Scope: Docs Effort: 1 pts 1/2-3 hours Status: To Do Can and should be resolved labels Nov 1, 2020
@J-Vaughan J-Vaughan changed the title [docs] detail how linting/commitlinting works [docs] detail how eslinting/commitlinting works Nov 1, 2020
@saihaj
Copy link
Member

saihaj commented Nov 1, 2020

This is in my list to work on. I added this as todo in summary of #26

saihaj added a commit that referenced this issue Nov 21, 2020
@saihaj saihaj linked a pull request Nov 21, 2020 that will close this issue
7 tasks
@saihaj saihaj linked a pull request Nov 21, 2020 that will close this issue
7 tasks
saihaj added a commit that referenced this issue Nov 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Effort: 1 pts 1/2-3 hours Scope: Docs Status: To Do Can and should be resolved
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants