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

[JENKINS-14900] [JENKINS-18662] [JENKINS-23444] Fixes for filter. #42

Merged
merged 7 commits into from Jul 20, 2014

Conversation

Projects
None yet
5 participants
@ikedam
Copy link
Member

ikedam commented Jul 13, 2014

This includes fixes for following issues:

  • JENKINS-14900: Copying from workspace drops files matching Ant's default excludes since Jenkins 1.456 (see also #12)
  • JENKINS-18662: Adding exclude filter
  • JENKINS-23444: Build doesn't fail even a file in "Artifacts to copy" doesn't exist. This is a designed behavior, and the problem is the inproper help texts.

Added "Artifacts not to copy" (excluded filter):
excludefilter

@cloudbees-pull-request-builder

This comment has been minimized.

Copy link

cloudbees-pull-request-builder commented Jul 13, 2014

plugins » copyartifact-plugin #82 SUCCESS
This pull request looks good

@jenkinsadmin

This comment has been minimized.

Copy link
Member

jenkinsadmin commented Jul 13, 2014

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

ikedam added a commit that referenced this pull request Jul 20, 2014

Merge pull request #42 from ikedam/feature/JENKINS-14900_Excludes
[JENKINS-14900] [JENKINS-18662] [JENKINS-23444] Fixes for filter.

@ikedam ikedam merged commit 13282b4 into jenkinsci:master Jul 20, 2014

@daniel-beck

This comment has been minimized.

Copy link
Member

daniel-beck commented Jul 23, 2014

Sorry I'm late commenting, but IMO not using default excludes when copying should be optional, with off being the default to prevent regressions. Compare jenkinsci/jenkins#1287

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.