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

Create .git-blame-ignore-revs file #1498

Merged
merged 6 commits into from
Nov 27, 2024
Merged

Conversation

StuAA78
Copy link
Contributor

@StuAA78 StuAA78 commented Nov 19, 2024

Our move away from Standard has involved a huge number of changes. In order to preserve the git blame record, we create a .git-blame-ignore-revs file, which we can use to ignore specific commits. We previously introduced it to our other repos when implementing Standard:

DEFRA/water-abstraction-team#37
DEFRA/water-abstraction-service#1746

This PR creates the file for this repo with the required commit to ignore the large-scale linting changes.

Note that while GitHub supports this file, it's up to us to configure our local repo to make use of it. This can be done with:

git config blame.ignorerevsfile .git-blame-ignore-revs

@StuAA78 StuAA78 added the housekeeping Refactoring, tidying up or other work which supports the project label Nov 19, 2024
@StuAA78 StuAA78 self-assigned this Nov 19, 2024
@Cruikshanks
Copy link
Member

Prettified!

@StuAA78 StuAA78 marked this pull request as ready for review November 27, 2024 15:54
@StuAA78 StuAA78 requested a review from Cruikshanks November 27, 2024 15:54
Copy link
Member

@Cruikshanks Cruikshanks left a comment

Choose a reason for hiding this comment

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

@StuAA78 StuAA78 merged commit afd1dd2 into main Nov 27, 2024
6 checks passed
@StuAA78 StuAA78 deleted the create-git-blame-ignore-revs branch November 27, 2024 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
housekeeping Refactoring, tidying up or other work which supports the project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants