Update mgmt yml file #6623
Merged
Update mgmt yml file #6623
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| <Import Project="Directory.Build.props" /> | ||
| <Import Project="Directory.Build.targets" /> | ||
| <Project Sdk="Microsoft.Build.Traversal"> | ||
| <ItemGroup> |
There was a problem hiding this comment.
@weshaggard won't it be easy for now until everything settles down where we have
instead of build.proj, we create
mgmt.proj
<takeYourPick>.proj (for non-mgmt projects)
There was a problem hiding this comment.
we have that which is eng/service.proj
|
/azp run net - mgmt - ci |
|
Azure Pipelines could not run because the pipeline triggers exclude this branch/path. |
|
/azp run net - mgmt - ci |
|
Azure Pipelines could not run because the pipeline triggers exclude this branch/path. |
|
@weshaggard I updated mgmt.yml last night as there were couple of PRs that wanted me to review/merge, so I excluded SDK directory, hence you see the conflict for that file. |
weshaggard
force-pushed
the
wesh/updatemgmtyml
branch
2 times, most recently
from
3979504 to
27e9f59
Compare
shahabhijeet
approved these changes
Jun 19, 2019
weshaggard
force-pushed
the
wesh/updatemgmtyml
branch
5 times, most recently
from
39fa9f5 to
4958579
Compare
Update mgmt yml file to be closer to client yml file still needs to be separate until we can get the mgmt tooling to work with dotnet pack/test commands. Added eng/mgmt.proj which builds all mangment libraries. Updated root build.proj to include both mgmt and client libraries. Remove init step from old mgmt.yml as it is needed any longer Add scope path validation Add support for publishing property
weshaggard
force-pushed
the
wesh/updatemgmtyml
branch
from
4958579 to
88f304b
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update mgmt yml file to be closer to client yml file
still needs to be separate until we can get the mgmt tooling
to work with dotnet pack/test commands.
Added eng/mgmt.proj which builds all mangment libraries.
Updated root build.proj to include both mgmt and client libraries.
PTAL @shahabhijeet @mitchdenny @chidozieononiwu