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

Application of RFC 2119 #87

Closed
hutson opened this issue Aug 25, 2018 · 8 comments · Fixed by #133
Closed

Application of RFC 2119 #87

hutson opened this issue Aug 25, 2018 · 8 comments · Fixed by #133

Comments

@hutson
Copy link
Contributor

hutson commented Aug 25, 2018

Re-evaluate how we are applying the terminology defined in RFC 2119 in the Conventional Commits specification.

Came up in #77 (comment)

@hutson
Copy link
Contributor Author

hutson commented Aug 25, 2018

cc @bcoe

@damianopetrungaro
Copy link
Member

@hbetts @bcoe any news on this?

@hutson
Copy link
Contributor Author

hutson commented Oct 16, 2018

I believe this issue is simply waiting on someone to have sufficient time to review the specification and ensure each use of MUST, SHOULD, etc., is being used correctly for the desired result.

@damianopetrungaro
Copy link
Member

Who wants to do it?
So I can assign it 😄

@damianopetrungaro
Copy link
Member

damianopetrungaro commented Feb 8, 2019

@hutson @zeke @raamdev @bcoe @stevemao who wants to pick this one?
I think it is pretty important for us.

@hutson
Copy link
Contributor Author

hutson commented Feb 10, 2019

I would love to, but I just can't commit to doing the work at this time 😢

@bcoe
Copy link
Contributor

bcoe commented Feb 18, 2019

@hutson @damianopetrungaro I'll make an effort to take this on. I essentially just need to re-read RFC 2119, go back through our spec, and correct the usage of our terminology I believe? i.e., we care about the "Specification" section and "Summary" section only?

@bcoe bcoe self-assigned this Feb 18, 2019
@damianopetrungaro
Copy link
Member

Yup @bcoe that's it :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants