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

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

Merged
merged 2 commits into from Mar 10, 2015

Conversation

Projects
None yet
4 participants
@bgruening
Copy link
Member

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'
Ignore list of files/directories that are specified in .shed.yml as "…
…shed_ignore". These files will not be included in the created tarball.
@jmchilton

This comment has been minimized.

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

This comment has been minimized.

Copy link
Member

martenson commented Mar 10, 2015

👍

@bgruening

This comment has been minimized.

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

Merge pull request #99 from bgruening/master
Ignore list of files/directories that are specified in .shed.yml

@jmchilton jmchilton merged commit 667270b into galaxyproject:master Mar 10, 2015

1 of 2 checks passed

coverage/coveralls Coverage pending from Coveralls.io
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@jmchilton

This comment has been minimized.

Copy link
Member

jmchilton commented Mar 10, 2015

Groovy gravy - thanks for the contribution!

@coveralls

This comment has been minimized.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment