Skip to content
This repository was archived by the owner on Mar 8, 2024. It is now read-only.

Commit 694ff83

Browse files
committed
Merge pull request #14 from github/lint-from-script-test
Mention linting in the script/test section
2 parents 3e924be + 24592a6 commit 694ff83

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

README.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,12 +74,15 @@ the application is up to date and can run appropriately.
7474
A good pattern to support is having an optional argument that is a file path.
7575
This allows you to support running single tests.
7676

77+
Linting (i.e. rubocop, jshint, pmd, etc.) can also be considered a form of testing. These tend to run faster than tests, so put them towards the beginning of a `script/test` so it fails faster if there's a linting problem.
78+
7779
`script/test` should be called from `script/cibuild`, so it should handle
7880
setting up the application appropriately based on the environment. For example,
7981
if called in a development environment, it should probably call `script/update`
8082
to always ensure that the application is up to date. If called from
8183
`script/cibuild`, it should probably reset the application to a clean state.
8284

85+
8386
### script/cibuild
8487

8588
`script/cibuild` is used for your continuous integration server.

0 commit comments

Comments
 (0)