Skip to content

Conversation

@keith
Copy link
Member

@keith keith commented Mar 25, 2025

Fixes #19753

@keith keith marked this pull request as draft March 25, 2025 20:04
@github-actions github-actions bot added the awaiting-review PR is awaiting review from an assigned reviewer label Mar 25, 2025
@keith keith force-pushed the ks/add-cc_import.defines branch from 7985795 to b1e52ae Compare March 25, 2025 20:15
@keith keith marked this pull request as ready for review March 26, 2025 00:42
@iancha1992 iancha1992 added the team-Rules-CPP Issues for C++ rules label Mar 26, 2025
@meteorcloudy meteorcloudy requested a review from pzembrod March 27, 2025 15:47
@meteorcloudy
Copy link
Member

@pzembrod can you take a look?

Copy link
Contributor

@pzembrod pzembrod left a comment

Choose a reason for hiding this comment

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

Sorry for the delay. I assume the branch will need a resync?

@keith keith force-pushed the ks/add-cc_import.defines branch from 27d2f40 to f1a447e Compare June 10, 2025 23:58
@keith keith force-pushed the ks/add-cc_import.defines branch from f1a447e to 0cfa1da Compare July 11, 2025 16:43
@keith
Copy link
Member Author

keith commented Jul 29, 2025

@pzembrod I think we're good here?

keith added a commit to keith/rules_cc that referenced this pull request Aug 14, 2025
This mirrors bazelbuild/bazel#25690 now that
this code lives in both places
@keith
Copy link
Member Author

keith commented Aug 14, 2025

hopefully helpful rules_cc pr bazelbuild/rules_cc#463

@keith keith requested a review from pzembrod October 17, 2025 18:25
@pzembrod pzembrod removed the awaiting-review PR is awaiting review from an assigned reviewer label Oct 31, 2025
@copybara-service copybara-service bot closed this in 4632c9e Nov 4, 2025
@keith keith deleted the ks/add-cc_import.defines branch November 19, 2025 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-Rules-CPP Issues for C++ rules

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cc_import should allow the 'defines' attribute

4 participants