Skip to content

Ignore list of files/directories that are specified in .shed.yml#99

Merged
jmchilton merged 2 commits intogalaxyproject:masterfrom
bgruening:master
Mar 10, 2015
Merged

Ignore list of files/directories that are specified in .shed.yml#99
jmchilton merged 2 commits intogalaxyproject:masterfrom
bgruening:master

Conversation

@bgruening
Copy link
Member

@bgruening bgruening commented Mar 10, 2015

The new section "ignore" can contain a list of files, directories or patterns that will not be included in the created tarball.
This is useful if you have some admin scripts included in this repository that are not intended for uploading into the Tool Shed.

Short example .shed.yml file:

# repository published to https://toolshed.g2.bx.psu.edu/repos/galaxyp/peptideshaker
owner: galaxyp
name: peptideshaker
ignore:
    - admin_scripts
    - '*.gz'

@bgruening
Ignore list of files/directories that are specified in .shed.yml as "…
cb1c335 
…shed_ignore". These files will not be included in the created tarball.
@jmchilton
Copy link
Member

jmchilton commented Mar 10, 2015

Awesome, thanks!

One small think - it is already called .shed.yml - so I feel like prefixing stuff with shed_ inside the file is redundant. Can we call the list just ignore instead?

I am happy to clean up the Travis linting problems and make this change post merge if you are okay with the change.

@martenson
Copy link
Member

martenson commented Mar 10, 2015

👍

@bgruening
Copy link
Member Author

bgruening commented Mar 10, 2015

@jmchilton I have changed the name and removed the white spaces. Will wait for Travis now :)

jmchilton added a commit that referenced this pull request Mar 10, 2015
@jmchilton
Merge pull request #99 from bgruening/master
667270b 
Ignore list of files/directories that are specified in .shed.yml
@jmchilton jmchilton merged commit 667270b into galaxyproject:master Mar 10, 2015
@jmchilton
Copy link
Member

jmchilton commented Mar 10, 2015

Groovy gravy - thanks for the contribution!

@coveralls
Copy link

coveralls commented Mar 10, 2015

Coverage Status

Coverage decreased (-0.21%) to 67.64% when pulling 1c3ae20 on bgruening:master into 1c6aea7 on galaxyproject:master.

@jmchilton jmchilton mentioned this pull request Apr 27, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@bgruening @jmchilton @martenson @coveralls