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

chore(format): executed make format on EVERYTHING (incl. tests) #1252

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

hilmarf
Copy link
Member

@hilmarf hilmarf commented Jan 14, 2025

everytime, I'm executing make format, those files get changed, even when they have nothing to do with the actual feature

@github-actions github-actions bot added kind/chore chore, maintenance, etc. area/documentation Documentation related component/ocm-cli OCM Command Line Interface size/s Small and removed kind/chore chore, maintenance, etc. labels Jan 14, 2025
@hilmarf hilmarf added this to the 2025-Q1 milestone Jan 14, 2025
@hilmarf hilmarf marked this pull request as ready for review January 14, 2025 09:21
@hilmarf hilmarf requested a review from a team as a code owner January 14, 2025 09:21
@hilmarf hilmarf enabled auto-merge (squash) January 14, 2025 09:22
@hilmarf hilmarf added kind/skip-release-notes Pull request will not appear in release notes area/testing Testing related labels Jan 14, 2025
@frewilhelm
Copy link
Contributor

Shouldn't we have an action to prevent code to be merged that is not correctly formatted? :O

@hilmarf hilmarf merged commit 2b764a2 into open-component-model:main Jan 14, 2025
27 checks passed
@hilmarf hilmarf deleted the chore/make_format_tests branch January 14, 2025 09:33
@hilmarf
Copy link
Member Author

hilmarf commented Jan 14, 2025

Shouldn't we have an action to prevent code to be merged that is not correctly formatted? :O

I believe we have that check, but it checks only on productive code and not on tests :-/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Documentation related area/testing Testing related component/ocm-cli OCM Command Line Interface kind/chore chore, maintenance, etc. kind/skip-release-notes Pull request will not appear in release notes size/s Small
Projects
Status: 🍺 Done
Development

Successfully merging this pull request may close these issues.

2 participants