Skip to content

chore: make root workspace buildable #1071

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

Merged
merged 2 commits into from
Apr 22, 2025
Merged

chore: make root workspace buildable #1071

merged 2 commits into from
Apr 22, 2025

Conversation

alexeagle
Copy link
Collaborator

replaces previous PR which tried to workaround this. Remove rules_lint - the reasoning is we want this repo to be simpler since it's a common dependency for users to visit, and also our code is in Go but rules_lint has no Go linters

@alexeagle alexeagle requested review from kormide and thesayyn April 22, 2025 00:22
@alexeagle alexeagle force-pushed the publish_bcr branch 2 times, most recently from f961f5d to e23787d Compare April 22, 2025 00:25
replaces previous PR which tried to workaround this.
Remove rules_lint - the reasoning is we want this repo to be simpler since it's a common dependency for users to visit, and also our code is in Go but rules_lint has no Go linters
Therefore there's a + (or earlier ~) character appended on the end of the value we compare against.
This seems undesirable.
@alexeagle alexeagle enabled auto-merge (squash) April 22, 2025 00:44
@alexeagle alexeagle merged commit c32593b into main Apr 22, 2025
31 checks passed
@alexeagle alexeagle deleted the publish_bcr branch April 22, 2025 00:45
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