Skip to content
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

Release pydoctor 24.3.0 #767

Merged
merged 25 commits into from
May 17, 2024
Merged

Release pydoctor 24.3.0 #767

merged 25 commits into from
May 17, 2024

Conversation

tristanlatr
Copy link
Contributor

@tristanlatr tristanlatr commented Mar 27, 2024

Fixes #794

Copy link

codecov bot commented Mar 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.62%. Comparing base (27f5d4d) to head (30c00ab).
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #767      +/-   ##
==========================================
+ Coverage   92.56%   92.62%   +0.05%     
==========================================
  Files          47       47              
  Lines        8393     8393              
  Branches     1850     1850              
==========================================
+ Hits         7769     7774       +5     
+ Misses        359      356       -3     
+ Partials      265      263       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

@tristanlatr tristanlatr marked this pull request as ready for review May 10, 2024 15:27
@tristanlatr
Copy link
Contributor Author

Releasing a new version of pydoctor @glyph @adiroiban.

This comment has been minimized.

This comment has been minimized.

1 similar comment

This comment has been minimized.

This comment has been minimized.

@adiroiban
Copy link
Member

Hi @tristanlatr I have pushed a change to publish the packagte using PyPI Trusted Publishers API

The PyPI changes were made here https://pypi.org/manage/project/pydoctor/settings/publishing/#errors

image


I think that only Twisted Organization admins have permissions to PyPI manager... but now, any package published from unit.yaml is accepted by PyPI.


Feel free to create a tag based on the latest commit from this PR.

It should trigger a PyPi package publishing.

Thanks

@adiroiban
Copy link
Member

I have also removed the PYPI_TOKEN token from secrets as it should never be used again

This comment has been minimized.

This comment has been minimized.

@adiroiban
Copy link
Member

I have removed the ubuntu 20.04 required jobs as it looks like it is no longer used.

image


@tristanlatr
Copy link
Contributor Author

Thanks a lot Adi, I’ll try again

@tristanlatr
Copy link
Contributor Author

Looks like it failed...

Error: Trusted publishing exchange failure: 
OpenID Connect token retrieval failed: GitHub: missing or insufficient OIDC token permissions, the ACTIONS_ID_TOKEN_REQUEST_TOKEN environment variable was unset

This generally indicates a workflow configuration error, such as insufficient
permissions. Make sure that your workflow has `id-token: write` configured
at the job level, e.g.:

``yaml
permissions:
  id-token: write
``

Learn more at https://docs.github.com/en/actions/deployment/security-hardening-your-deployments/about-security-hardening-with-openid-connect#adding-permissions-settings.

Full log: https://github.com/twisted/pydoctor/actions/runs/9098901922/job/25010411600

@adiroiban

This comment has been minimized.

@adiroiban
Copy link
Member

True. My bad.

I have added explicit job permissions.

You can delete the tag and create it again. It should re-trigger the job

This comment has been minimized.

@tristanlatr
Copy link
Contributor Author

Still not working @adiroiban

ERROR    HTTPError: 400 Bad Request from https://upload.pypi.org/legacy/        
         Can't have direct dependency: twisted@                                 
         git+https://github.com/twisted/twisted.git ; extra == "mypy". See      
         https://packaging.python.org/specifications/core-metadata for more     
         information.  

This comment has been minimized.

@adiroiban
Copy link
Member

The pypi publishing works.

The package was rejected due to setup.cfg deps definition

twisted @ git+https://github.com/twisted/twisted.git


It looks like we need to do a release for Twisted.

Why do we need the latest Twisted here ?

@tristanlatr
Copy link
Contributor Author

I think we don’t need it anymore

And fix the twisted intersphinx as a drive-by change.

This comment has been minimized.

This comment has been minimized.

Copy link

According to pydoctor_primer, this change doesn't affect pydoctor warnings on a corpus of open source code. ✅

1 similar comment
Copy link

According to pydoctor_primer, this change doesn't affect pydoctor warnings on a corpus of open source code. ✅

Copy link

According to pydoctor_primer, this change doesn't affect pydoctor warnings on a corpus of open source code. ✅

@tristanlatr tristanlatr merged commit dca9625 into master May 17, 2024
23 checks passed
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.

Release automation is broken
2 participants