Skip to content

Conversation

rickeylev
Copy link
Collaborator

Spurred by the discussion to converge on using . to separate generated targets, I
wrote down some of the conventions we've adopted.

@rickeylev rickeylev requested a review from aignas as a code owner April 23, 2025 22:11
@rickeylev rickeylev requested a review from dougthor42 April 23, 2025 22:12
Copy link
Collaborator

@dougthor42 dougthor42 left a comment

Choose a reason for hiding this comment

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

Couple questions but nothing blocking. LGTM.

@aignas aignas enabled auto-merge April 29, 2025 13:57
@aignas aignas added this pull request to the merge queue Apr 29, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 29, 2025
@rickeylev rickeylev added this pull request to the merge queue Apr 29, 2025
github-merge-queue bot pushed a commit that referenced this pull request Apr 29, 2025
Spurred by the discussion to converge on using `.` to separate generated
targets, I
wrote down some of the conventions we've adopted.

---------

Co-authored-by: Ignas Anikevicius <[email protected]>
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 29, 2025
@rickeylev rickeylev added this pull request to the merge queue Apr 29, 2025
Merged via the queue into bazel-contrib:main with commit 189e30d Apr 29, 2025
3 checks passed
@rickeylev rickeylev deleted the docs.style branch April 29, 2025 17:19
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.

3 participants