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

feat: support multiple scopes and display_scopes option #46

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

farfromrefug
Copy link

This PR adds support for multiple scopes are per conventional-commit specs.
I split using "," then for display i join using ", ".
I also added display_scopes to disable displaying of scopes in the changelog

@xotahal
Copy link
Owner

xotahal commented Jun 18, 2024

Could you point us to conventional-commit spec where they talk about multiple scopes, please?

@farfromrefug
Copy link
Author

@xotahal in fact it is not in https://www.conventionalcommits.org/en/v1.0.0/ ! . I found out about it in conventional-changelog/commitlint#901 and i though they added it because it was in the specs.
So do as you want and add it or not. I will use it in my fork cause i really need it.
Just to explain the reason why, i use it to filter ios/android/desktop. I can get something like core(ios,collectionview): ... and so when generating the changelog i can filter out ios when i am building for android

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.

2 participants