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

PCT updates #129

Merged
merged 10 commits into from Jan 31, 2018

Conversation

Projects
None yet
3 participants
@jglick
Copy link
Member

jglick commented Jan 4, 2018

POM updates plus failures I noticed running against newer cores.

@reviewbybees esp. @oleg-nenashev

jglick added some commits Jan 4, 2018

EnvInjectMigrationBuildWrapperTest.testMigratePreSandboxJobInSecuredJ…
…enkins did not run against Jenkins 2.x due to missing matrix-auth detached dependency.

@jglick jglick changed the title POM updates PCT updates Jan 4, 2018

jglick added some commits Jan 4, 2018

Revert "Ought to have corrected Jenkins 2 regressions now."
This reverts commit c399a17.
Without push permission, I cannot verify the effect of such a change.
@oleg-nenashev
Copy link
Member

oleg-nenashev left a comment

🐝 AFAICT

@jglick

This comment has been minimized.

Copy link
Member Author

jglick commented Jan 4, 2018

@reviewbybees

This comment has been minimized.

Copy link

reviewbybees commented Jan 11, 2018

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

@oleg-nenashev oleg-nenashev merged commit 5d05b71 into jenkinsci:master Jan 31, 2018

1 check passed

continuous-integration/jenkins/pr-merge This commit looks good
Details

@jglick jglick deleted the jglick:pom branch Jan 31, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.