-
-
Notifications
You must be signed in to change notification settings - Fork 59
Workflow/pipeline support #10
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
base: main
Are you sure you want to change the base?
Conversation
…ped Jenkins core version dependency and refactored where required)
…es. Moved help files to webapp folder for common access.
throw AbortException in SimpleBuildStep perform to notify failed builds
|
Cool. Thanks a lot for the PR. I'll look at it in a few days. I have to setup a 2.0 with workflow setup. |
|
Awesome work, finally there is a working pipeline support. But since your changes variables in the installation dir are not getting parsed anymore.
|
|
Where is I'm not sure where to access build variables for pipeline jobs, as the build type is now a The offending code is likely here. Any fixes are appreciated. |
|
Have you had time to look into this yet? |
oleg-nenashev
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good AFAICT
|
Would actually be really neat to have to change in there ;) |


This PR adds support to run the unity3d plugin via pipeline scripts.
e.g.
Contains support for the snippet generator:
This required bumping up the required jenkins plugins dependency version to 1.642.3. This changed some unit tests to require
MasterToSlaveCallablebecause of different security restrictions.A dependency to
org.jenkins-ci.plugins.workflow.workflow-step-apihas been added.This mainly adds the
Unity3dBuilderStepclass and changesUnity3dBuilderto implement theSimpleBuildStepinterface. This changes the perform method to require aRun<?, ?>instead of anAbstractBuild<?, ?>. ARundoes not have access to a workspace, so aFilePathis injected for alternative access to paths andComputers.I've moved out some methods to a helper class for use by the Unity3dBuilderStep class.
The only issue I have left is a compiler warning that I can't seem to get rid of:
Warning unknown enum constant edu.umd.cs.findbugs.annotations.When.ANYTIME Unity3dBuilder.java C:/Repos/unity3d-plugin/src/main/java/org/jenkinsci/plugins/unity3d/Unity3dBuilder.javaIt seems to be an issue with the
com.google.guavadependency version being updated. This had to be raised due to the InjectedTest suite changing from the 1.642.3 jenkins plugins version.Let me know if there's anything I should take a look at or improve.