Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Python 3.9#1264

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
filmor merged 6 commits intomasterfrompython3.9
Nov 13, 2020
Merged

Python 3.9#1264

filmor merged 6 commits intomasterfrompython3.9
Nov 13, 2020

Conversation

filmor
Copy link
Member

What does this implement/fix? Explain your changes.

Adds Python 3.9 to CI.

Checklist

Check all those that are applicable and complete.

  • Make sure to include one or more tests for your change
  • If an enhancement PR, please create docs and at best an example
  • Add yourself toAUTHORS
  • Updated theCHANGELOG

@filmor
Copy link
MemberAuthor

We'll have to wait until 3.9 is available in both Travis and Appveyor.

@filmor
Copy link
MemberAuthor

@domdfcoding
Copy link

Python 3.9 is now available on Travis

- Only record structs when they are defined, not when they are declared- If a struct was only declared when a typedef was created, it won't  contain its member declarations. Those have to be drawn from the  recorded structs instead.- Rename internal members of AstParser to make it easier to debug
@filmorfilmor changed the titleAdd Python 3.9 to CIPython 3.9Nov 13, 2020
@filmor
Copy link
MemberAuthor

@lostmsu This is passing tests now, feel free to review. I'll backport this to 2.5.

domdfcoding reacted with thumbs up emoji

@codecov-io
Copy link

codecov-io commentedNov 13, 2020
edited
Loading

Codecov Report

Merging#1264 (a1316a1) intomaster (36bdb98) willincrease coverage by0.34%.
The diff coverage isn/a.

Impacted file tree graph

@@            Coverage Diff             @@##           master    #1264      +/-   ##==========================================+ Coverage   87.62%   87.97%   +0.34%==========================================  Files           1        1                Lines         291      291              ==========================================+ Hits          255      256       +1+ Misses         36       35       -1
FlagCoverage Δ
setup_linux65.29% <ø> (+0.34%)⬆️
setup_windows74.22% <ø> (+0.34%)⬆️

Flags with carried forward coverage won't be shown.Click here to find out more.

Impacted FilesCoverage Δ
setup.py87.97% <0.00%> (+0.34%)⬆️

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last update36bdb98...a1316a1. Read thecomment docs.

Copy link
Member

@lostmsulostmsu left a comment

Choose a reason for hiding this comment

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

Looks good, but maybe removing underscore from most members of gen_interop is a bit destructive to file's blame.

I'd also squash.

@filmor
Copy link
MemberAuthor

I'll clean things up a bit, but if I ever need to debug this again, I want the double-underscores already gone ;)

@filmorfilmor merged commit6be3636 intomasterNov 13, 2020
filmor added a commit that referenced this pull requestNov 13, 2020
* Add Python 3.9 to CI* Update AppVeyor image and always install all requirements* Add Python 3.9 interop file* Fix geninterop script and regenerate interop39.cs    - Only record structs when they are defined, not when they are declared    - If a struct was only declared when a typedef was created, it won't      contain its member declarations. Those have to be drawn from the      recorded structs instead.   - Rename internal members of AstParser to make it easier to debug
@filmorfilmor deleted the python3.9 branchJanuary 23, 2021 11:30
@retheon
Copy link

Hi, it seems this branch is already merged to master but herehere it says that pythonnet does not support python 3.9. Does this page have to be updated or there is still a step I'm missing before python 3.9 is supported?

@filmor
Copy link
MemberAuthor

Things are complicated, check#1389.

retheon reacted with thumbs up emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@lostmsulostmsulostmsu approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

6 participants
@filmor@domdfcoding@codecov-io@retheon@lostmsu@sschukat

[8]ページ先頭

©2009-2025 Movatter.jp