Skip to content
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

rule_test: apply "tags" to all rules in the macro #8784

Closed
wants to merge 3 commits into from

Conversation

laszlocsomor
Copy link
Contributor

RELNOTES: rule_test: fix Bazel 0.27 regression ("tags" attribute was ingored, #8723

Fixes #8723

Change-Id: I25ac338e4978084085b8c49d6d0a1c47d8dc4fd1

RELNOTES: rule_test: fix Bazel 0.27 regression ("tags" attribute was ingored, bazelbuild#8723

Fixes bazelbuild#8723

Change-Id: I25ac338e4978084085b8c49d6d0a1c47d8dc4fd1
Change-Id: Iee65afe1b7b8d3f2e09057804249396193ed241b
@laszlocsomor
Copy link
Contributor Author

Oh no, I noticed this breaks down with test-specific attributes (e.g. shard_count), which are passed also to the non-test rule the macro expands to.

Let me fix this and I'll ping this thread.

Change-Id: Ie53b8787ab8679ff4896820ef552c2696eac1d09
@laszlocsomor
Copy link
Contributor Author

Done.

@bazel-io bazel-io closed this in 6d0b14b Jul 4, 2019
@laszlocsomor laszlocsomor deleted the rule_test branch July 4, 2019 08:30
siberex pushed a commit to siberex/bazel that referenced this pull request Jul 4, 2019
RELNOTES: rule_test: fix Bazel 0.27 regression ("tags" attribute was ingored, bazelbuild#8723

Fixes bazelbuild#8723

Change-Id: I25ac338e4978084085b8c49d6d0a1c47d8dc4fd1

Closes bazelbuild#8784.

Change-Id: Ie53b8787ab8679ff4896820ef552c2696eac1d09
PiperOrigin-RevId: 256509917
laurentlb pushed a commit that referenced this pull request Jul 8, 2019
RELNOTES: rule_test: fix Bazel 0.27 regression ("tags" attribute was ingored, #8723

Fixes #8723

Change-Id: I25ac338e4978084085b8c49d6d0a1c47d8dc4fd1

Closes #8784.

Change-Id: Ie53b8787ab8679ff4896820ef552c2696eac1d09
PiperOrigin-RevId: 256509917
laurentlb pushed a commit that referenced this pull request Jul 8, 2019
RELNOTES: rule_test: fix Bazel 0.27 regression ("tags" attribute was ingored, #8723

Fixes #8723

Change-Id: I25ac338e4978084085b8c49d6d0a1c47d8dc4fd1

Closes #8784.

Change-Id: Ie53b8787ab8679ff4896820ef552c2696eac1d09
PiperOrigin-RevId: 256509917
irengrig pushed a commit to irengrig/bazel that referenced this pull request Jul 15, 2019
RELNOTES: rule_test: fix Bazel 0.27 regression ("tags" attribute was ingored, bazelbuild#8723

Fixes bazelbuild#8723

Change-Id: I25ac338e4978084085b8c49d6d0a1c47d8dc4fd1

Closes bazelbuild#8784.

Change-Id: Ie53b8787ab8679ff4896820ef552c2696eac1d09
PiperOrigin-RevId: 256509917
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rule_test does not use tags since bazel 0.27
3 participants