Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@JohnVillalovos
Copy link
Member

@JohnVillalovos JohnVillalovos commented Oct 23, 2024

Also fix __annotations__ not working in Python 3.14 by using the
annotation on the 'class' instead of on the 'instance'

Closes: #3013

@JohnVillalovos JohnVillalovos marked this pull request as draft October 23, 2024 04:56
@JohnVillalovos JohnVillalovos marked this pull request as ready for review October 23, 2024 04:59
@JohnVillalovos JohnVillalovos requested a review from nejch October 23, 2024 05:01
@JohnVillalovos
Copy link
Member Author

@nejch This ended up being easier than I thought it would be.

@codecov
Copy link

codecov bot commented Oct 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.62%. Comparing base (3739b5d) to head (642ea08).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3023      +/-   ##
==========================================
- Coverage   96.62%   96.62%   -0.01%     
==========================================
  Files          95       95              
  Lines        6101     6099       -2     
==========================================
- Hits         5895     5893       -2     
  Misses        206      206              
Flag Coverage Δ
api_func_v4 82.62% <100.00%> (+0.10%) ⬆️
cli_func_v4 82.91% <100.00%> (-0.01%) ⬇️
unit 88.75% <100.00%> (-0.01%) ⬇️

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

Files with missing lines Coverage Δ
gitlab/base.py 99.50% <100.00%> (-0.01%) ⬇️

Also fix __annotations__ not working in Python 3.14 by using the
annotation on the 'class' instead of on the 'instance'

Closes: #3013
@nejch
Copy link
Member

nejch commented Oct 23, 2024

Well that was easy :) thanks @JohnVillalovos

@nejch nejch merged commit 14d2a82 into main Oct 23, 2024
18 checks passed
@nejch nejch deleted the jlvillal/py314 branch October 23, 2024 15:46
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.

Python 3.14 breaks unit tests

3 participants