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

Restore commit-lock-file-summary rename for consistency #10988

Merged
merged 2 commits into from
Jun 29, 2024

Conversation

cole-h
Copy link
Member

@cole-h cole-h commented Jun 28, 2024

It was originally renamed in #10691, but #9063 accidentally removed the new name and alias.

Motivation

Context

Priorities and Process

Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@cole-h cole-h requested a review from edolstra as a code owner June 28, 2024 18:03
@cole-h cole-h mentioned this pull request Jun 28, 2024
@cole-h cole-h marked this pull request as draft June 28, 2024 19:26
@cole-h

This comment was marked as resolved.

@cole-h cole-h force-pushed the restore-summary-rename branch from 7421f94 to d906d42 Compare June 28, 2024 21:54
@github-actions github-actions bot added the with-tests Issues related to testing. PRs with tests have some priority label Jun 28, 2024
@cole-h
Copy link
Member Author

cole-h commented Jun 28, 2024

OK, test added as the first commit. For me, it fails when I check out that commit, and passes when I check out the tip of this branch.

I also noticed a little weirdness with aliases to settings and filed a bug here: #10989. Maybe this is intended / already has an open issue prior to mine / has been discussed before; if so, feel free to point me there (but I'll make the argument that if the "real" setting works as a flag (i.e. --commit-lock-file-summary), its aliases should also work as a flag (i.e. --commit-lockfile-summary).

@cole-h cole-h marked this pull request as ready for review June 28, 2024 21:56
Copy link
Member

@roberth roberth left a comment

Choose a reason for hiding this comment

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

Thank you Cole, and the test is much appreciated ❤️

@roberth roberth merged commit 32e6cc6 into NixOS:master Jun 29, 2024
11 checks passed
@cole-h cole-h deleted the restore-summary-rename branch June 29, 2024 14:58
rvolosatovs added a commit to rvolosatovs/nix-flake-update-action that referenced this pull request Jul 12, 2024
NixOS/nix#10691
NixOS/nix#10988

Signed-off-by: Roman Volosatovs <rvolosatovs@riseup.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
with-tests Issues related to testing. PRs with tests have some priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants