Explicitly include test file in STRTA #26
Merged
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.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
After templating a new project, I noticed that
scripts/test
was missing from the scaffolded project. Turns out, g8 ignores files namedtest
by default! A full list of defaults is available here.This was addressed with the introduction of the
.g8ignore
file, which declares files to be excluded from the templated project. By declaring a negation rule forscripts/test
, we override the default and include the file.This regression was introduced in #10.
Testing Instructions
sbt new azavea/azavea.g8
and accept the defaults.ls quickstart/scripts
.test
will be missing.quickstart
project withrm -rf quickstart
.sbt new file:///path/to/local/azavea.g8
.ls quickstart/scripts
.test
should be present.