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

Expand "Make" variables inside of values in the env dictionary on test rules #3088

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aaronsky
Copy link

This PR implements a gap introduced when bazelbuild/rules_apple#2476 was introduced, where expandable env settings on test rules would not be expanded into the xcscheme during generation. There are a couple of things that reviewers should be aware of before proceeding with this change:

  • This change is not considerate of the rules_apple being used, meaning that it assumes the developer is using at least version 3.6.0 (when the breaking changes in Fix label encoding in incremental generators #2476 were introduced). This is not good and should be considered a blocker, but I don't know how to address this with the tools offered by the ruleset or Bazel.
  • This change as-implemented requires regeneration of the xcschemes if the configured value of the "Make" variable ever changes.

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.

1 participant