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

Add environment variables support #80

Merged
merged 4 commits into from
Jun 4, 2019

Conversation

kshtsk
Copy link
Contributor

@kshtsk kshtsk commented Feb 8, 2019

Signed-off-by: Kyr [email protected]

@oleg-nenashev oleg-nenashev self-requested a review February 8, 2019 15:46
@kshtsk
Copy link
Contributor Author

kshtsk commented Feb 12, 2019

What is the next?

@oleg-nenashev
Copy link
Member

@kshtsk the plugin is not actively maintained now. I requested my review, but it may take a while
https://groups.google.com/d/msg/jenkinsci-dev/uc6NsMoCFQI/AIO4WG1UCwAJ

Copy link
Member

@basil basil left a comment

Choose a reason for hiding this comment

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

Looks good to me! The integration tests I wrote in #81 haven't been merged yet, but once they are I would like to see some integration tests for this change written before it is merged.

@basil
Copy link
Member

basil commented May 21, 2019

#81 has been merged, so we now have a realistic test framework for this plugin. Can you please write an integration test for this functionality? Once that is done, this should be ready to merge.

@basil
Copy link
Member

basil commented Jun 4, 2019

I wrote an integration test and did some minor code cleanup, so this should be ready to go now.

@basil basil merged commit 75cb611 into jenkinsci:master Jun 4, 2019
@basil
Copy link
Member

basil commented Feb 13, 2020

This was just released in Swarm 3.18.

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.

3 participants