Skip to content

Conversation

leon-xd
Copy link
Contributor

@leon-xd leon-xd commented Feb 19, 2025

As discussed a bit ago, added a link to the conventional commits spec in the CONTRIBUTING.md file.
Also reworded and cleaned up the whole Submitting a Pull Request section.

@leon-xd leon-xd requested a review from wmmc88 February 19, 2025 23:36
* Commit your changes using a descriptive commit message.
* Push your commit to your forked repository.
* In GitHub, create a pull request to merge your fork to `main`.
* Make sure to use the [Conventional Commits](https://www.conventionalcommits.org) specification
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if we should enforce this automatically?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe with https://github.com/marketplace/actions/semantic-pull-request? Not a blocker for this pr but worth considering

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I could do this for this PR - sounds like a pretty easy fix and it's within scope. Will take a bit longer as we should discuss the full set of types that we would like to allow

@leon-xd leon-xd changed the title Added conventional commits spec to CONTRIBUTING.md chore: Added conventional commits spec to CONTRIBUTING.md Feb 20, 2025
@wmmc88 wmmc88 marked this pull request as draft February 22, 2025 05:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants