Skip to content

Add bazel and run copyright and format check#2

Closed
lurtz wants to merge 4 commits intomainfrom
check-and-fix-licences-via-bazel
Closed

Add bazel and run copyright and format check#2
lurtz wants to merge 4 commits intomainfrom
check-and-fix-licences-via-bazel

Conversation

@lurtz
Copy link

@lurtz lurtz commented Feb 5, 2026

The same checks are still enforced via pre-commit hooks, which run the bazel targets instead.

note: I have to change the year to 2025 to make the check pass

@lurtz lurtz force-pushed the check-and-fix-licences-via-bazel branch from 1f53ba4 to 5a74c67 Compare February 5, 2026 14:11
@lurtz lurtz changed the title add bazel and comment opengrep Add bazel and run copyright and format check Feb 5, 2026
@opajonk
Copy link

opajonk commented Feb 5, 2026

I am not sure what to say... yes, consistency is also a value. On the other hand, I do not think that using Bazel as a tool integration system is a reasonable way forward for "any and all tools". It seems overly complex for such a trivial thing as "execute this tool".

@lurtz lurtz force-pushed the check-and-fix-licences-via-bazel branch from df69f67 to 3c79814 Compare February 6, 2026 11:24
@lurtz lurtz changed the base branch from check-and-fix-licences to main February 6, 2026 11:24
@lurtz lurtz closed this Feb 6, 2026
@lurtz lurtz deleted the check-and-fix-licences-via-bazel branch February 6, 2026 11:37
@lurtz lurtz restored the check-and-fix-licences-via-bazel branch February 6, 2026 11:38
@lurtz
Copy link
Author

lurtz commented Feb 6, 2026

continued in eclipse-score#79

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