Skip to content

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

Merged
merged 35 commits into from
Nov 1, 2019
Merged
Changes from 33 commits
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
ae791f0
Merge pull request #1 from TheAlgorithms/master
mrvnmchm Aug 17, 2019
ae675f0
Merge pull request #2 from TheAlgorithms/master
mrvnmchm Sep 13, 2019
32f1435
Merge pull request #3 from TheAlgorithms/master
mrvnmchm Oct 3, 2019
670f700
Merge pull request #4 from TheAlgorithms/master
mrvnmchm Oct 30, 2019
8c6574f
travis test
mrvnmchm Oct 30, 2019
cc55035
travis pull ID test
mrvnmchm Oct 31, 2019
5f12b5f
get pr branch test
mrvnmchm Oct 31, 2019
59145de
Merge branch 'master' into travis-test
mrvnmchm Oct 31, 2019
17d19d7
Merge pull request #5 from mrvnmchm/travis-test
mrvnmchm Oct 31, 2019
abbf642
retry pr build
mrvnmchm Oct 31, 2019
22a12c9
test pushing back - probable git error for origin 'not found'
mrvnmchm Oct 31, 2019
654c99f
Merge pull request #6 from mrvnmchm/travis-test
mrvnmchm Oct 31, 2019
c463129
github auth?
mrvnmchm Oct 31, 2019
3ec0c04
Merge pull request #7 from mrvnmchm/travis-test
mrvnmchm Oct 31, 2019
6391d72
add .sh
mrvnmchm Oct 31, 2019
b5f7305
chmod
mrvnmchm Oct 31, 2019
916970f
Merge pull request #8 from mrvnmchm/travis-test
mrvnmchm Oct 31, 2019
ac50a2c
add index update for permission fix
mrvnmchm Oct 31, 2019
0cb7bf9
Merge pull request #9 from mrvnmchm/travis-test
mrvnmchm Oct 31, 2019
938b395
Merge pull request #10 from mrvnmchm/travis-test
mrvnmchm Oct 31, 2019
d895a00
run sh for script
mrvnmchm Oct 31, 2019
0ac923e
add all
mrvnmchm Oct 31, 2019
5e63965
Merge pull request #11 from mrvnmchm/travis-test
mrvnmchm Oct 31, 2019
4322e39
Merge pull request #12 from mrvnmchm/travis-test
mrvnmchm Oct 31, 2019
bbe8bd3
add pull directory
mrvnmchm Oct 31, 2019
184585c
Merge pull request #13 from mrvnmchm/travis-test
mrvnmchm Oct 31, 2019
dd4dd4e
fetch pr branch
mrvnmchm Oct 31, 2019
78b7dab
swap placement of adding commits
mrvnmchm Oct 31, 2019
b0eaef3
Merge pull request #15 from mrvnmchm/travis-test
mrvnmchm Oct 31, 2019
92df8b9
rotate
mrvnmchm Oct 31, 2019
bb63b2f
quit trying to update Travis
mrvnmchm Oct 31, 2019
6a9e608
Merge pull request #16 from mrvnmchm/travis-test
mrvnmchm Oct 31, 2019
6c39148
Merge pull request #17 from TheAlgorithms/master
mrvnmchm Oct 31, 2019
b28fdf6
formatting leftovers
mrvnmchm Oct 31, 2019
126b3a4
Travis CI: Write & print DIRECTORY.md on one line
cclauss Nov 1, 2019
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions .travis.yml
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
Copy link
Member

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?

Copy link
Member Author

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.

install: pip install -r requirements.txt
before_script:
- black --check . || true
Expand All @@ -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
Copy link
Member

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?

Copy link
Member Author

Choose a reason for hiding this comment

The 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