Skip to content

docs: add CONTRIBUTING guidelines#38

Merged
tbouffard merged 1 commit intomainfrom
docs/add_contributing_guide
Jan 28, 2025
Merged

docs: add CONTRIBUTING guidelines#38
tbouffard merged 1 commit intomainfrom
docs/add_contributing_guide

Conversation

@tbouffard
Copy link
Member

These guidelines are used by default in repositories that don't include specific guidelines.

These guidelines are used by default in repositories that don't include specific guidelines.
@tbouffard tbouffard added the documentation Improvements or additions to documentation label Jan 28, 2025

Branch naming is essential for understanding the purpose of the branch and eventually, for automatically adding labels to the PR.

### Branch Naming Convention
Copy link
Member Author

Choose a reason for hiding this comment

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

thought: We have specific convention for bpmn-visualization.
I suggest we apply the same conventions in all repositories and use the one describe here as it is commonly used in a large amount of projects.
Do you agree @csouchet?

Current bpmn-visualization convention: https://github.com/process-analytics/bpmn-visualization-js/blob/24963c8693d5dec600c7d4288f523c4ef068ea62/CONTRIBUTING.md#fork--create-a-branch

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, it's fine for me.

Copy link
Member Author

Choose a reason for hiding this comment

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

OK, I am working on a new PR

@tbouffard tbouffard marked this pull request as ready for review January 28, 2025 13:28
@tbouffard tbouffard merged commit 0177149 into main Jan 28, 2025
2 checks passed
@tbouffard tbouffard deleted the docs/add_contributing_guide branch January 28, 2025 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants