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

Jenkinsfile: remove Windows stages, remove hack/ci/windows.ps1 #43672

Merged
merged 1 commit into from
Jun 1, 2022

Conversation

thaJeztah
Copy link
Member

These are now running in GitHub actions, so we no longer need to run them in Jenkins.

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

byebye

These are now running in GitHub actions, so we no longer need to
run them in Jenkins.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
# Jenkins CI scripts for Windows to Windows CI (Powershell Version)
# By John Howard (@jhowardmsft) January 2016 - bash version; July 2016 Ported to PowerShell

$ErrorActionPreference = 'Stop'
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing this file as well, as GitHub actions isn't using it

Copy link
Member

@neersighted neersighted left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM / good riddance 😄

Copy link
Member

@crazy-max crazy-max left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thaJeztah
Copy link
Member Author

Alright; let's merge this one!

@thaJeztah thaJeztah merged commit 3e32104 into moby:master Jun 1, 2022
@thaJeztah thaJeztah deleted the remove_windows_jenkins branch June 1, 2022 09:02
@thaJeztah thaJeztah changed the title Jenkinsfile: remove Windows stages Jenkinsfile: remove Windows stages, remove hack/ci/windows.ps1 Jun 1, 2022
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.

None yet

5 participants