Skip to content

Conversation

@MiYanni
Copy link
Contributor

@MiYanni MiYanni commented Dec 18, 2018

This PR changes the building of modules so that they output to an artifacts folder at the root of the repo. This removes the output folder being src/Package/<CONFIG>/ResourceManager/AzureResourceManager to artifacts/<CONFIG>. The test artifacts also go to artifacts/Test now.

This PR requires Common changes found here: Azure/azure-powershell-common#107

…d all references to src/Package/<CONFIG>/ResourceManager/AzureResourceManager/ that I could find. Removed legacy/unused files or logic that used this old output pathing.
…cenario tests. Fixed a few other places to use artifacts folder. Using artifacts/Test for test artifacts.
@MiYanni
Copy link
Contributor Author

MiYanni commented Dec 19, 2018

@MiYanni
Copy link
Contributor Author

MiYanni commented Dec 20, 2018

@MiYanni
Copy link
Contributor Author

MiYanni commented Dec 20, 2018

I attempted to output the nupkgs (from a signed build) to artifacts/Package but was unsuccessful. If that change is desired, it can be addressed in a different PR. For now, the nupkgs will be output to artifacts.

# Conflicts:
#	src/Az.props
#	tools/GenerateHelp.ps1
#	tools/StaticAnalysis/Exceptions/Az.Aks/MissingAssemblies.csv
@MiYanni
Copy link
Contributor Author

MiYanni commented Dec 27, 2018

@MiYanni
Copy link
Contributor Author

MiYanni commented Dec 27, 2018

Needs common change prior to merging: Azure/azure-powershell-common#107

@MiYanni
Copy link
Contributor Author

MiYanni commented Dec 28, 2018

@markcowl markcowl merged commit a57ed45 into Azure:master Jan 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants