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

Add helper script snippet for following renames #1135

Merged
merged 1 commit into from
Feb 20, 2025

Conversation

mschoettle
Copy link
Contributor

@mschoettle mschoettle commented Jan 28, 2025

The current example does not follow renames which I think should be included to know the very first time the file was introduced.
For some reason, git log --reverse --follow does not work properly. So reversed it to use tail instead of head.

I added an extra section but it might be worth to add it to the existing examples instead.

  • Added self to AUTHORS.rst.
  • My changes do not contradict
    the current specification.
  • I agree to license my contribution under the licenses indicated in the
    changed files.
  • Added a change log entry in changelog.d/<directory>/.

I feel that the changes are too small and do not affect the tool itself, so I did not add one. If you disagree, I'll be happy to add a changelog entry.

  • Added self to copyright blurb of touched files.

@mschoettle
Copy link
Contributor Author

@floriansnow @carmenbianca Can one of you please review?

@floriansnow
Copy link
Contributor

Thank you for your contribution! This looks good and useful to me! I agree that this does not warrant an entry in the changelog, but please add your copyright notice in the changed file. We want to track authors properly, even if it's a small change.

@mschoettle
Copy link
Contributor Author

@floriansnow Thanks for the feedback. Added the copyright notice to the file.

@floriansnow floriansnow merged commit 5074e95 into fsfe:main Feb 20, 2025
15 checks passed
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.

None yet

2 participants