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

cmd/cueckoo: add short commit hash to "Closes" for importpr #35

Merged
merged 1 commit into from
Mar 16, 2023

Conversation

myitcv
Copy link
Member

@myitcv myitcv commented Mar 16, 2023

(see commit message)

For #33.

@myitcv myitcv requested a review from mvdan March 16, 2023 11:46
When we import a PR from GitHub we immediately lose track of the
original commit sha because of necessary commit message rewriting.
Currently this means we have no easy way of anyone tracking exactly what
was imported.

Per a suggestion from @mvdan, we can simply add this to the "Closes"
message for now, and later consider writing a message back to the PR
from which we are importing with more information.

For #33.
@myitcv myitcv force-pushed the add_abbrev_commit_to_closes_message branch from 27c1ae5 to 43a7b09 Compare March 16, 2023 11:49
@mvdan mvdan merged commit fa87e85 into master Mar 16, 2023
@mvdan mvdan deleted the add_abbrev_commit_to_closes_message branch March 16, 2023 15:04
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