-
Notifications
You must be signed in to change notification settings - Fork 33
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
Contributing and checking commits #13
Contributing and checking commits #13
Conversation
059d8da
to
b722e1a
Compare
29140fa
to
414aaa6
Compare
Hmm, Looks like conventional commit doesn't like more than 1 colon in commit message title - https://github.com/TexasInstruments/processor-sdk-doc/actions/runs/11524739733/attempts/1#summary-32085557172 |
6219fb2
to
e508d78
Compare
The bots looks super strict - https://github.com/StaticRocket/processor-sdk-doc/actions/runs/11526783806/attempts/1#summary-32091486204 Has a restriction in branch name as well |
I must say, I LOVE IT! |
Yeah, but that's more for filtering branches on the primary repo, which there shouldn't really be anything other than master unless we decide to do some massive work that requires multiple contributions. |
Probably not on this commit, but we should consider adding something like https://blog.alta3.com/topics/scm-and-pipelines/adding-spell-checking-as-a-github-action/ -> I wonder if github enterprise has a paid service for grammar and spell check |
e508d78
to
5041f6f
Compare
Add a short file detailing how one can contribute to the project. Signed-off-by: Randolph Sapp <[email protected]>
Add a commit check for PRs to make sure they are following the rules outlined in the CONTRIBUTING document. Signed-off-by: Randolph Sapp <[email protected]>
5041f6f
to
67d52da
Compare
I was referring to the branch names it expects from the author creating PRs as well. |
Also, please edit $subject line of PR: s/Contibuting/Contributing so that the merge message looks clean. |
I was thinking about that, but considering all the acronyms and odd names we'd definitely need to make our own dictionary for it to be useful. That being said, if we actually did make an acronym and odd word dict, it would make for good appendix fodder. |
Add a short file detailing how one can contribute to the project.
Add a commit check for PRs to make sure they are following the rules outlined in the CONTRIBUTING document.
Resolves #6
Resolves #10