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.

Sign up for GitHub

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

Jump to bottom

Build script tidy-up #922

Merged
merged 6 commits into from
Feb 13, 2022
Merged

Build script tidy-up #922

merged 6 commits into from
Feb 13, 2022

Conversation

martincostello
Copy link
Member

  • Do not edit the csproj outside CI to help prevent stopping people checking in changes from local testing with the build scripts.
  • Use xplat path joining.
  • Fix some casing.
  • Make brace styles/spacing consistent.
  • Add shebang for PowerShell Core.
  • Set executable bit for PowerShell Core.
  • Remove redundant variable.
  • Disable showing the download progress bar when downloading files.
  • Skip running the tests for .NET Framework on non-Windows operating systems.
  • Drop test TFM for .NET Core 2.1 as it is no longer supported.
  • Speed up CI by setting some additional environment variables.

@martincostello
Tidy up build scripts
4f6371e 
Do not edit the csproj outside CI.
Use xplat path joining.
Fix some casing.
Make brace styles/spacing consistent.
Add shebang for pwsh.
Remove redundant variable.
@martincostello
Set executable bit
1e27c32 
Set the executable bit for macOS and Linux.
@martincostello
Disable download progress
5bd5fa6 
Disable showing the download progress bar when downloading files.
@martincostello
Skip net4xx tests on non-Windows
53996cf 
Skip running the tests for .NET Framework on non-Windows operating systems.
@martincostello
Drop test target for netcoreapp2.1
eb4e407 
.NET Core 2.1 is no longer supported.
@martincostello
Speed up CI
874188a 
Speed up CI by setting some additional environment variables.
@martincostello martincostello added this to the v7.2.4 milestone Feb 13, 2022
@martincostello martincostello merged commit 440955d into App-vNext:v724-or-v730 Feb 13, 2022
@martincostello martincostello deleted the Cake-Tidy branch February 13, 2022 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
CI/build
Projects
None yet
Milestone
v7.2.4
Development

Successfully merging this pull request may close these issues.
1 participant
@martincostello