Disable cue_consolidated_instance generation by default#10
Merged
Conversation
Add cue_gen_consolidated_instance directive to opt-in to generating cue_consolidated_instance rules. Previously these were always generated, now they require explicit opt-in via the gazelle directive. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Fix the comment for cue_gen_exported_instance directive to match its actual name. Allow both cue_gen_exported_instance and cue_gen_consolidated_instance directives to be explicitly disabled with "false" value, enabling subdirectories to override parent config. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…irective Enable cue_gen_consolidated_instance in root BUILD.bazel and disable it in services/api/ to show per-subdirectory override with "false" value. Update README with directive documentation. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…-out Verified that gazelle does not regenerate the rule when cue_gen_consolidated_instance is set to false. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…tance Updated comments to clarify behavior of cueGenExportedInstance and cueGenConsolidatedInstance.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
cue_gen_consolidated_instancegazelle directive to opt-in to generatingcue_consolidated_instancerules. Previously these were always generated, now they require explicit opt-in.falsevalue to disable the directive in subdirectories (e.g.,# gazelle:cue_gen_consolidated_instance false), allowing per-package control.cue_gen_exported_instancedirective to match its actual name.cue_gen_exported_instance.examples/gazelle/to demonstrate both opt-in at root and opt-out inservices/api/.Usage
Test plan
config/andcontacts/retaincue_consolidated_instancerules with directive enabledservices/api/does not regeneratecue_consolidated_instancewith directive set tofalsebazel run //:gazelleinexamples/gazelle/to confirm correct behavior