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

Fix PR validation builds against forks no. 2 #6259

Merged
merged 3 commits into from Mar 9, 2022
Merged

Conversation

BruceHaley
Copy link
Contributor

@BruceHaley BruceHaley commented Mar 9, 2022

Fixes #minor

This fixes a regression from PR #6227.

The nuget restore task fails when a nuget.config file is not created. So instead of skipping nuget.config creation for forks, this now points sources to nuget.org for forks.

Also fixes mac PR builds against forks.

@BruceHaley BruceHaley requested a review from a team as a code owner March 9, 2022 20:00
@BruceHaley BruceHaley changed the title Fix PR validation builds against forks Fix PR validation builds against forks no. 2 Mar 9, 2022
@BruceHaley BruceHaley added the Automation: No parity PR does not need to be applied to other languages. label Mar 9, 2022
@mrivera-ms mrivera-ms merged commit f7b481e into main Mar 9, 2022
@mrivera-ms mrivera-ms deleted the bruce/configfix3-9 branch March 9, 2022 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Automation: No parity PR does not need to be applied to other languages.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants