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

Update the versions in the templates to the real values #1912

Merged
merged 3 commits into from
Aug 3, 2021

Conversation

mattleibow
Copy link
Member

Description of Change

Use the version that we built against in the templates.

@mattleibow mattleibow added this to the 6.0.100-preview.7 milestone Aug 3, 2021
@mattleibow mattleibow requested a review from Eilon as a code owner August 3, 2021 00:46
@mattleibow mattleibow changed the base branch from main to release/6.0.1xx-preview7 August 3, 2021 00:47
@mattleibow mattleibow changed the title Add the template to the sample (#1867) Update the versions in the templates to the real values Aug 3, 2021
Comment on lines +38 to +39
.Replace('MICROSOFT_EXTENSIONS_VERSION_VALUE', '$(MicrosoftExtensionsFileProvidersEmbeddedPackageVersion)')
.Replace('PROJECT_REUNION_VERSION_VALUE', '$(_MicrosoftProjectReunionVersion)'))" />
Copy link
Member Author

Choose a reason for hiding this comment

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

The hackiest way, but basically replace the values in the json files with the props values.

@Eilon
Copy link
Member

Eilon commented Aug 3, 2021

The logic looks legit, but does it leave the files changed on disk? What I did in another project where I had a similar problem is that the checked-in files were called template.in.json (example) and the MSBuild did the edits and saved them as template.json, and there was also a .gitignore (example) in the folder to make sure the output files didn't get checked in.

@mattleibow
Copy link
Member Author

I am working in the obj directory, so we should be safe.

Copy link
Member

@rmarinho rmarinho left a comment

Choose a reason for hiding this comment

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

  D:\agent\1\s\bin\dotnet\sdk\6.0.100-preview.7.21379.14\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.targets(137,5): error NETSDK1085: The 'NoBuild' property was set to true but the 'Build' target was invoked. [D:\agent\1\s\src\Templates\src\Microsoft.Maui.Templates.csproj]

@mattleibow mattleibow merged commit f6c3cc7 into release/6.0.1xx-preview7 Aug 3, 2021
@mattleibow mattleibow deleted the dev/replace-template-versions branch August 3, 2021 16:11
@github-actions github-actions bot locked and limited conversation to collaborators Dec 25, 2023
@samhouts samhouts added the fixed-in-6.0.100-preview.7.3 Look for this fix in 6.0.100-preview.7.3! label Aug 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
fixed-in-6.0.100-preview.7.3 Look for this fix in 6.0.100-preview.7.3!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants