Skip to content

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.

2 participants