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

Allow constants to request a root area different than Constants #168

Merged
merged 1 commit into from
Jan 25, 2023

Conversation

kzu
Copy link
Member

@kzu kzu commented Jan 25, 2023

This opens up other scenarios for specific constants we don't want behind a Constants propery (i.e ThisAssembly.Git, see #69).

A small refactoring of the targets with a new PrepareConstants will also make it easier for extenders to run before we do our conversion to AdditionalFiles, to keep the extended items clean even when they need to run before/after certain targets.

@kzu kzu added the enhancement New feature or request label Jan 25, 2023
@kzu kzu enabled auto-merge (rebase) January 25, 2023 05:40
This opens up other scenarios for specific constants we don't want
behind a Constants propery (i.e ThisAssembly.Git, see #69).

A small refactoring of the targets with a new PrepareConstants
will also make it easier for extenders to run before we do our
conversion to AdditionalFiles, to keep the extended items clean
even when they need to run before/after certain targets.
@kzu kzu force-pushed the dev/ConstantsRoot branch from 462d3ab to 5892aff Compare January 25, 2023 05:56
@kzu kzu merged commit 9589ddb into main Jan 25, 2023
@kzu kzu deleted the dev/ConstantsRoot branch January 25, 2023 06:01
@devlooped devlooped locked and limited conversation to collaborators Sep 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant