Skip to content

Conversation

@vvv444
Copy link

@vvv444 vvv444 commented Aug 26, 2015

https://issues.jenkins-ci.org/browse/JENKINS-30141

User can choose the desired behavior now (like --depth and --set-depth flags in svn)

User can choose the desired behavior now (like --depth and --set-depth flags in svn)
@jenkinsadmin
Copy link
Member

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

@recena recena changed the title Option to choose if depth is sticky [JENKINS-30141] Option to choose if depth is sticky Aug 26, 2015
Copy link
Member

Choose a reason for hiding this comment

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

binary compatibility issue. It's better to keep and deprecate the original constructor

Copy link
Author

Choose a reason for hiding this comment

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

Thank you! I'm quite new to Java :-)
I've pushed a fix commit to my repository. Now, how is it customary to
proceed in such cases? Shall I open a new pull request?

On Wed, Sep 9, 2015 at 12:32 PM, Oleg Nenashev notifications@github.com
wrote:

In src/main/java/hudson/scm/SubversionSCM.java
#134 (comment)
:

     }

     @DataBoundConstructor
  •    public ModuleLocation(String remote, String credentialsId, String local, String depthOption, boolean ignoreExternalsOption) {
    
  •    public ModuleLocation(String remote, String credentialsId, String local, String depthOption, boolean stickyDepth, boolean ignoreExternalsOption) {
    

binary compatibility issue. It's better to keep and deprecate the original
constructor


Reply to this email directly or view it on GitHub
https://github.com/jenkinsci/subversion-plugin/pull/134/files#r39023278.

Copy link
Member

Choose a reason for hiding this comment

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

Could you also create a help file for this new option?

Copy link
Author

Choose a reason for hiding this comment

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

Sure. Done :-)

Copy link
Contributor

Choose a reason for hiding this comment

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

@vvv444 Please, use <code> instead of <tt>. I'll try to merge this PR soon.

Copy link
Author

Choose a reason for hiding this comment

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

done

@vvv444
Copy link
Author

vvv444 commented Oct 12, 2015

Please note the check failed due to some timeout problem. Not because of my code. How can I rerun it?

@recena
Copy link
Contributor

recena commented Oct 13, 2015

@vvv444 close and open the PR.

@vvv444 vvv444 closed this Oct 13, 2015
@vvv444 vvv444 reopened this Oct 13, 2015
@vvv444
Copy link
Author

vvv444 commented Feb 3, 2016

@recena @oleg-nenashev Hey guys, any reason why this wasn't integrated?

@recena
Copy link
Contributor

recena commented Feb 3, 2016

@vvv444 Yes, I'm avoiding to include new features until we improve our test coverage and solve bugs related to svn:externals. I'm sorry for that.

@jglick
Copy link
Member

jglick commented Jun 16, 2017

I would also prefer to avoid new features unless they are obviously self-contained like SCMBrowsers.

@rshah88
Copy link

rshah88 commented Aug 28, 2017

Hey guys, any ETA on when will this feature be integrated ?

@oleg-nenashev
Copy link
Member

Somebody need to resolve the merge conflict

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.

7 participants