Skip to content

Add GitHub action for PR creation and Wix file generation logic - #12748

Merged
Aditya Patwardhan (adityapatwardhan) merged 6 commits into
PowerShell:masterfrom
adityapatwardhan:AutoUpdateDotnet
May 22, 2020
Merged

Add GitHub action for PR creation and Wix file generation logic#12748
Aditya Patwardhan (adityapatwardhan) merged 6 commits into
PowerShell:masterfrom
adityapatwardhan:AutoUpdateDotnet

Conversation

@adityapatwardhan

@adityapatwardhan Aditya Patwardhan (adityapatwardhan) commented May 21, 2020

Copy link
Copy Markdown
Member

PR Summary

When we update to a new version of .NET we need to update the files.wxs file as well. This change automates that. The script needs to be run on Windows for that though.

Also add GitHub Action to create PR when there is an update, scheduled for 13:00 UTC. (6 AM PST now)

PR Context

PR Checklist

Comment thread tools/UpdateDotnetRuntime.ps1
Comment thread tools/UpdateDotnetRuntime.ps1 Outdated
@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) marked this pull request as draft May 21, 2020 22:49
Comment thread .github/workflows/daily.yml Outdated
@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) marked this pull request as ready for review May 22, 2020 01:05
@adityapatwardhan Aditya Patwardhan (adityapatwardhan) changed the title Add logic to create the Wix packaging file when updating .NET previews Add GitHub action for PR creation and Wix file generation logic May 22, 2020
@adityapatwardhan

Copy link
Copy Markdown
Member Author

PoshChan-Bot (@PoshChan) Please remind me in 1 hour

@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) merged commit 0e435a5 into PowerShell:master May 22, 2020
@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) deleted the AutoUpdateDotnet branch May 22, 2020 01:47
@PoshChan

Copy link
Copy Markdown
Collaborator

Aditya Patwardhan (@adityapatwardhan), this is the reminder you requested 1 hour ago

@bergmeister

Christoph Bergmeister (bergmeister) commented May 22, 2020

Copy link
Copy Markdown
Contributor

Aditya Patwardhan (@adityapatwardhan) This seems to create a PR for each fork and is going to create a lot of noise, is it possible to be opt-in or apply only to the PowerShell organisation itself?
bergmeister#4
The discussion here suggests to add an if conditional on the job: if: github.repository == 'PowerShell/PowerShell'
I tried to create a PR here: #12763

@adityapatwardhan

Copy link
Copy Markdown
Member Author

Christoph Bergmeister (@bergmeister) Thank you for finding this and creating a PR for the fix.

@adityapatwardhan Aditya Patwardhan (adityapatwardhan) added the CL-Tools Indicates that a PR should be marked as a tools change in the Change Log label May 22, 2020
@ghost

Copy link
Copy Markdown

🎉v7.1.0-preview.4 has been released which incorporates this pull request.:tada:

Handy links:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-Tools Indicates that a PR should be marked as a tools change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants