Skip to content
This repository has been archived by the owner on Nov 10, 2022. It is now read-only.

Ability to define an svnignore file for svn-push #168

Closed
frankiejarrett opened this issue Apr 19, 2016 · 2 comments
Closed

Ability to define an svnignore file for svn-push #168

frankiejarrett opened this issue Apr 19, 2016 · 2 comments

Comments

@frankiejarrett
Copy link
Contributor

It would be uh-mazing if an .svnignore file could be supported in the plugin root and honored during svn-push. It would allow developers to ignore anything dev-related that they don't want in the WordPress.org repo.

Currently, it's left solely up to dev-lib to magically decide what is dev and what isn't.
#167 is related.

@westonruter
Copy link
Contributor

Yes, this is a good idea. Right now the list of excludes is hard-coded:

wp-dev-lib/svn-push

Lines 126 to 144 in 0bcb8c9

--exclude '/.david-dev' \
--exclude '/package.json' \
--exclude '/contributing.md' \
--exclude '/composer.json' \
--exclude '/Gruntfile.js' \
--exclude '/readme.md' \
--exclude '/.ci-env.sh' \
--exclude '/.dev-lib' \
--exclude '/.coveralls.yml' \
--exclude '/.git*' \
--exclude '/.jscsrc' \
--exclude '/.jshint*' \
--exclude '/.travis.yml' \
--exclude '/assets' \
--exclude '/dev-lib' \
--exclude '/svn-url' \
--exclude '/tests' \
--exclude 'phpunit.xml*' \
--exclude 'phpcs.ruleset.xml' \

    --exclude '/.david-dev' \
    --exclude '/package.json' \
    --exclude '/contributing.md' \
    --exclude '/composer.json' \
    --exclude '/Gruntfile.js' \
    --exclude '/readme.md' \
    --exclude '/.ci-env.sh' \
    --exclude '/.dev-lib' \
    --exclude '/.coveralls.yml' \
    --exclude '/.git*' \
    --exclude '/.jscsrc' \
    --exclude '/.jshint*' \
    --exclude '/.travis.yml' \
    --exclude '/assets' \
    --exclude '/dev-lib' \
    --exclude '/svn-url' \
    --exclude '/tests' \
    --exclude 'phpunit.xml*' \
    --exclude 'phpcs.ruleset.xml' \

There are two exclusion-related arguments for rsync:

     --exclude=PATTERN       exclude files matching PATTERN
     --exclude-from=FILE     read exclude patterns from FILE

So I think supporting this would be as simple as checking to see if .svnignore exists, and if so, use it as path for --exclude-file instead of manually listing out each --exclude.

@westonruter
Copy link
Contributor

This is really now obsolete now as svn-push is deprecated in favor of grunt-wp-deploy.

And besides, the .svnignore file is in itself obsolete in favor of .distignore: #202.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants