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

Added development files and directories to bower ignore. #1063

Merged
merged 1 commit into from
Oct 28, 2015
Merged

Added development files and directories to bower ignore. #1063

merged 1 commit into from
Oct 28, 2015

Conversation

jdufresne
Copy link
Contributor

No description provided.

@jdufresne
Copy link
Contributor Author

I have modified the commit to now exclude everything except bower.json and dist. Thanks.

@Mottie
Copy link
Owner

Mottie commented Oct 28, 2015

Thanks for double checking 👍

Mottie added a commit that referenced this pull request Oct 28, 2015
Added development files and directories to bower ignore.
@Mottie Mottie merged commit e284dd6 into Mottie:master Oct 28, 2015
Mottie added a commit that referenced this pull request Nov 2, 2015
@Mottie
Copy link
Owner

Mottie commented Nov 2, 2015

I had to revert this change, and I ended up just adding all the folders to ignore as you did initially... now the dist folder appears empty sigh

@jdufresne
Copy link
Contributor Author

I believe this is because you didn't anchor the directories at the root directory. So js and css directories inside /dist will be excluded. Try the following. The leading slash indicates the file is anchored at the root of the repository.

"ignore": [
    "/node_modules",
    "/bower_components",
    "/.git",
    "/addons",
    "/beta-testing",
    "/css",
    "/docs",
    "/js",
    "/testing",
    "/Gruntfile.js",
    "/package.json",
    "*.txt",
    "*.md",
    "*.html",
    "/tablesorter.jquery.json"
]

Mottie added a commit that referenced this pull request Nov 5, 2015
@Mottie
Copy link
Owner

Mottie commented Nov 5, 2015

Ok, it looks like that fixed it. Thanks!

@jdufresne
Copy link
Contributor Author

Thanks!

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

Successfully merging this pull request may close these issues.

2 participants