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

Support strings in manifest versions #17329

Merged

Conversation

dsplaisted
Copy link
Member

Port of #16746 to 5.0.3xx

@dsplaisted dsplaisted requested review from mhutch, wli3, sfoslund and a team April 30, 2021 20:11
@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

Copy link
Member

@sfoslund sfoslund left a comment

Choose a reason for hiding this comment

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

LGTM once the failing test gets updated.

@@ -4,67 +4,72 @@
<body>
<trans-unit id="ConcreteWorkloadHasNoDescription">
<source>Workload definition '{0}' is a concrete dev workload but has no description</source>
<target state="translated">워크로드 정의 '{0}'이(가) 구체적인 개발 워크로드이지만 설명이 없습니다.</target>
<target state="new">Workload definition '{0}' is a concrete dev workload but has no description</target>
Copy link
Member

Choose a reason for hiding this comment

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

Looks like we're losing some translations here

mhutch and others added 3 commits April 30, 2021 16:18
Check dependencies in workload manifests (dotnet#16746)

* Implement workload manifest dependency validation

* Test workload manifest dependency validation

* Workload manifest version is now semver

* Update projects that link/reference WorkloadManifestReader
@dsplaisted dsplaisted force-pushed the workload-manifest-string-version branch from 34d1d74 to ff83aa9 Compare April 30, 2021 23:19
@dsplaisted
Copy link
Member Author

@mhutch Note the fix to FXVersion.ToString in ff83aa9

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.

3 participants