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

Lift requirement for Issue-Ref in commit message #137

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

Conversation

LittleHuba
Copy link
Contributor

Commits are linked by Github to the PR that introduced them. It is impossible to merge a commit without PR.
Hence, it is sufficient if the PR links to issues.

We have no tooling that would verify that the linkage of a commit is correct. Thus, making an issue reference in the commit message mandatory holds no benefit.

It is still allowed to reference an issue in a commit message. It is simply no longer mandatory.

Issue-ref: closes #131
Issue-ref: closes #119

Commits are linked by Github to the PR that introduced them.
It is impossible to merge a commit without PR.
Hence, it is sufficient if the PR links to issues.

We have no tooling that would verify that the linkage of a
commit is correct. Thus, making an issue reference in the
commit message mandatory holds no benefit.

It is still allowed to reference an issue in a commit
message. It is simply no longer mandatory.

Issue-ref: closes #131
Issue-ref: closes #119
Copy link

Copy link

@LittleHuba LittleHuba marked this pull request as ready for review December 20, 2024 13:25
@nradakovic
Copy link
Contributor

This PR doesn't closes #131. Please read all points of acceptance criteria. This PR just solves one point but it doesn't cover whole scope. Adjust PR description to reflect this.

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