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

Revert "Use official Jenkins CI to build and test winp" #128

Merged
merged 1 commit into from
Aug 7, 2024

Conversation

basil
Copy link
Member

@basil basil commented Aug 7, 2024

In general, we do not accept regressions. Prior to #116, the repository was in a state such that changes to native code were not releasable but changes to non-native code (such as #69 and #127) were releasable. #116 (in the absence of #117) regressed the ability to release changes to non-native code, but this temporary situation seemed acceptable under the assumption that #117 would be completed shortly thereafter. Since #117 has not been completed in a timely fashion, I am reverting #116 (and the subsequent two fixup commits) to fix this regression. It can be reintegrated (by reverting this reversion) once the fix for #117 has been prepared.

Testing done

@basil basil added the bug label Aug 7, 2024
@basil basil merged commit 56f77dc into jenkinsci:master Aug 7, 2024
13 checks passed
@basil basil added skip-changelog and removed bug labels Aug 7, 2024
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.

1 participant