Skip to content

update CI to include Python 3.9rc1#432

Closed
nitzmahone wants to merge 1 commit into
yaml:release/5.4from
nitzmahone:py39rc1_ci_update
Closed

update CI to include Python 3.9rc1#432
nitzmahone wants to merge 1 commit into
yaml:release/5.4from
nitzmahone:py39rc1_ci_update

Conversation

@nitzmahone
Copy link
Copy Markdown
Member

No description provided.

@nitzmahone nitzmahone force-pushed the py39rc1_ci_update branch 5 times, most recently from b07ab22 to 9663319 Compare August 31, 2020 18:10
@nitzmahone nitzmahone changed the base branch from master to release/5.4 August 31, 2020 18:36
@cclauss
Copy link
Copy Markdown
Contributor

cclauss commented Sep 2, 2020

Travis CI build config validation issues...
Screenshot 2020-09-02 at 08 08 33


# ensure required-for-build packages are present and up-to-date
Invoke-Exe { & $python -W "ignore:DEPRECATION" -m pip install --upgrade cython wheel setuptools --no-warn-script-location }
Invoke-Exe { & $python -W "ignore:DEPRECATION" -m pip install --no-warn-script-location --upgrade cython wheel setuptools --no-warn-script-location }
Copy link
Copy Markdown
Contributor

@cclauss cclauss Sep 2, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is doing --no-warn-script-location twice on one line.

Should we really be suppressing deprecation warnings?

@webknjaz
Copy link
Copy Markdown

webknjaz commented Sep 5, 2020

@cclauss those are not issues, just informational messages. They aren't even warnings.

@cclauss
Copy link
Copy Markdown
Contributor

cclauss commented Sep 5, 2020

I know what they are and why they are there.
Providing the same command line option twice in one command is a source of confusion.

@webknjaz
Copy link
Copy Markdown

webknjaz commented Sep 5, 2020

@cclauss I was responding to your Travis comment

Comment thread .travis.yml
env: TOXENV=py38
- python: 3.8-dev
env: TOXENV=py38
- python: 3.9-dev
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Python 3.9 final is now available:

Suggested change
- python: 3.9-dev
- python: 3.9

Comment thread .travis.yml
arch: arm64
env: TOXENV=py38
- python: 3.8-dev
- python: 3.9-dev
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- python: 3.9-dev
- python: 3.9

@ingydotnet
Copy link
Copy Markdown
Member

Closing.

I believe this PR is covered by #407

Reopen if I missed something.

@ingydotnet ingydotnet closed this Dec 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants