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

ci: fail if standalone.py changed and standalone.tpl did not #206

Merged
merged 1 commit into from
Nov 22, 2024

Conversation

leseb
Copy link
Collaborator

@leseb leseb commented Nov 22, 2024

1d45668 ci: fail if standalone.py changed and standalone.tpl did not

commit 1d45668
Author: Sébastien Han [email protected]
Date: Fri Nov 22 10:03:33 2024 +0100

ci: fail if standalone.py changed and standalone.tpl did not

This commit adds to the pre-commit hook a check to verify that changes
to standalone.py are accompanied by corresponding updates to
standalone.tpl. The check will fail if standalone.py is updated without
updating standalone.tpl ONLY if no components file(s) is/are updated.
This ensures consistency between standalone.py and standalone.tpl, and
prompts the user to run 'make standalone' if necessary. The check can be
bypassed by adding the 'ignore-standalone-change' label to the pull
request.

Signed-off-by: Sébastien Han <[email protected]>

@leseb leseb force-pushed the ci-fail-standalone-py-only-change branch 5 times, most recently from cc1e077 to d68ad8e Compare November 22, 2024 09:28
This commit adds to the pre-commit hook a check to verify that changes
to standalone.py are accompanied by corresponding updates to
standalone.tpl. The check will fail if standalone.py is updated without
updating standalone.tpl ONLY if no components file(s) is/are updated.
This ensures consistency between standalone.py and standalone.tpl, and
prompts the user to run 'make standalone' if necessary. The check can be
bypassed by adding the 'ignore-standalone-change' label to the pull
request.

Signed-off-by: Sébastien Han <[email protected]>
@leseb leseb force-pushed the ci-fail-standalone-py-only-change branch from d68ad8e to ad04b81 Compare November 22, 2024 09:31
Copy link
Member

@tumido tumido left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

❤️ Love it!

@tumido tumido merged commit 7df7f5a into opendatahub-io:main Nov 22, 2024
1 check passed
@leseb leseb deleted the ci-fail-standalone-py-only-change branch November 22, 2024 10:29
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