-
-
Notifications
You must be signed in to change notification settings - Fork 46.6k
Travis CI: Write & print DIRECTORY.md on one line #1542
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
Changes from 33 commits
ae791f0
ae675f0
32f1435
670f700
8c6574f
cc55035
5f12b5f
59145de
17d19d7
abbf642
22a12c9
654c99f
c463129
3ec0c04
6391d72
b5f7305
916970f
ac50a2c
0cb7bf9
938b395
d895a00
0ac923e
5e63965
4322e39
bbe8bd3
184585c
dd4dd4e
78b7dab
b0eaef3
92df8b9
bb63b2f
6a9e608
6c39148
b28fdf6
126b3a4
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,8 @@ | ||
language: python | ||
python: 3.7 | ||
cache: pip | ||
before_install: pip install --upgrade pip setuptools | ||
before_install: | ||
- pip install --upgrade pip setuptools | ||
install: pip install -r requirements.txt | ||
before_script: | ||
- black --check . || true | ||
|
@@ -11,5 +12,4 @@ script: | |
- mypy --ignore-missing-imports . | ||
- pytest . --doctest-modules | ||
after_success: | ||
- scripts/build_directory_md.py > DIRECTORY.md | ||
- cat DIRECTORY.md | ||
- scripts/build_directory_md.py 2>&1 | tee DIRECTORY.md | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why this change? How is it different than the current code? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This change eliminates a line and does both operations. Printing the output, and writing it to the file, see tee |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why this change? How is it different than the current code?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Formatting leftovers during previous attempts of testing builds. I'll revert this.