Skip to content

Using checkMethod on c:select#192

Merged
jglick merged 3 commits intojenkinsci:masterfrom
jglick:checkMethod
Sep 8, 2017
Merged

Using checkMethod on c:select#192
jglick merged 3 commits intojenkinsci:masterfrom
jglick:checkMethod

Conversation

@jglick
Copy link
Copy Markdown
Member

@jglick jglick commented Aug 8, 2017

Downstream of jenkinsci/credentials-plugin#90.

@reviewbybees

Comment thread pom.xml
<jenkins.version>1.609.3</jenkins.version>
<java.level>6</java.level>
<jenkins.version>1.625.3</jenkins.version>
<java.level>7</java.level>
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

forced by credentials update

@ghost
Copy link
Copy Markdown

ghost commented Aug 8, 2017

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.

@jglick jglick requested a review from stephenc August 15, 2017 12:41
Copy link
Copy Markdown

@ghost ghost left a comment

Choose a reason for hiding this comment

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

🐝

@jglick
Copy link
Copy Markdown
Member Author

jglick commented Aug 16, 2017

@reviewbybees done

Copy link
Copy Markdown
Member

@stephenc stephenc left a comment

Choose a reason for hiding this comment

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

Pending upstream release

@jglick jglick merged commit cccaf24 into jenkinsci:master Sep 8, 2017
@jglick jglick deleted the checkMethod branch September 8, 2017 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants