Skip to content

removing legacy CI files - #1783

Merged
tsnoam merged 1 commit into
masterfrom
delete_legacy_files
Feb 20, 2020
Merged

removing legacy CI files#1783
tsnoam merged 1 commit into
masterfrom
delete_legacy_files

Conversation

@Poolitzer

Copy link
Copy Markdown
Member

No description provided.

@tsnoam
tsnoam merged commit 0fb0fbb into master Feb 20, 2020
@tsnoam
tsnoam deleted the delete_legacy_files branch February 20, 2020 15:33
@septatrix

Copy link
Copy Markdown
Contributor

Searching in the source code still yields some references to travis and appveyor. Should these also be removed?

$ grep --line-number --ignore-case appveyor -R *
CHANGES.rst:621:- Unitests are now also running on AppVeyor (Windows VM).
tests/test_jobqueue.py:41:@pytest.mark.skipif(os.getenv('APPVEYOR'), reason="On Appveyor precise timings are not accurate.")
tests/test_messagequeue.py:28:@pytest.mark.skipif(os.getenv('APPVEYOR'), reason="On Appveyor precise timings are not accurate.")
tests/test_bot.py:495:    @pytest.mark.skipif(os.getenv('APPVEYOR') and (sys.version_info < (3, 6)),
tests/test_bot.py:496:                        reason='only run on 3.6 on appveyor')

$ grep --line-number --ignore-case travis -R *
CHANGES.rst:157:- travis.yaml: TEST_OFFICIAL removed from allowed_failures.
CHANGES.rst:161:- Add Python 3.8 (RC version) to Travis testing matrix (`#1543`_).
tests/test_updater.py:147:        port = randrange(1024, 49152)  # Select random port for travis
tests/test_updater.py:183:        port = randrange(1024, 49152)  # Select random port for travis
tests/test_updater.py:207:        port = randrange(1024, 49152)  # Select random port for travis
tests/test_updater.py:227:        port = randrange(1024, 49152)  # Select random port for travis
tests/test_updater.py:282:        port = randrange(1024, 49152)  # select random port for travis
tests/conftest.py:35:TRAVIS = os.getenv('TRAVIS', False)
tests/conftest.py:37:if TRAVIS:
tests/conftest.py:38:    pytest_plugins = ['tests.travis_fold']
tests/travis_fold.py:32:        terminalreporter.write('travis_fold:start:plugin.{}\n{}\n'.format(plugin_name, text))
tests/travis_fold.py:34:        terminalreporter.write('travis_fold:end:plugin.{}\n'.format(plugin_name))
tests/travis_fold.py:81:        terminal.write('\ntravis_fold:start:{}\r'.format(name.split('::')[1]))
tests/travis_fold.py:82:        terminal.write('travis_time:start:{}time\r'.format(name.split('::')[1]))
tests/travis_fold.py:92:            terminal.write('\n\ntravis_fold:end:{}'.format(name.split('::')[1]))
tests/travis_fold.py:93:        terminal.write('\rtravis_time:end:{}time:'
tests/travis_fold.py:96:        time.sleep(0.001)  # Tiny sleep so travis hopefully doesn't mangle the log

@Bibo-Joshi

Copy link
Copy Markdown
Member

Yes, I think they can be removed as well (except the lines from the changelog, of course). Would you care to pr?

tsnoam commented Feb 23, 2020

Copy link
Copy Markdown
Member

@Bibo-Joshi Travis fold might still be used.

Copy link
Copy Markdown
Member

@tsnoam The config.py only uses it when tests are running on Travis

tsnoam commented Feb 23, 2020

Copy link
Copy Markdown
Member

@Bibo-Joshi In that case, I have no objections to remove :)

@github-actions github-actions Bot locked and limited conversation to collaborators Aug 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants