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

Output build artifacts to artifacts folder #8159

Merged
merged 16 commits into from
Jan 2, 2019

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