Update Jekyll.gitignore #2540
Merged
Update Jekyll.gitignore #2540
Conversation
Add .jekyll-cache/ as per envygeeks/jekyll-assets
|
If github/pages-gem#189 is applied, this change will be neccesary |
|
@tasken this seems like a pretty safe change irrespective of the linked issue. Thanks! |
dellagustin
added a commit
to dellagustin/innersourcecommons.org
that referenced
this pull request
Feb 21, 2020
While implementing the docker support for development environment setup (InnerSourceCommons#91) I noticed that the `.jekyll-cache` folder was created, and this folder was not on `.gitignore`. I could not find documentation about it, but some references that it should be excluded form git: - github/gitignore@ea2aa40#diff-3b5726570b981bf3e11c93a6e7bc3ff9 - github/gitignore#2540 It does not seem like content that belongs to the repo, so I added to the ignore list. I wonder if other people also have this folder being generated, or if it is a side-effect of the Docker setup somehow.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Reasons for making this change:
Add .jekyll-cache/ as per https://github.com/envygeeks/jekyll-assets
Links to documentation supporting these rule changes: