Skip to content

Update package metadata for all packages #1259

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

Closed
srikanthccv opened this issue Sep 1, 2022 · 14 comments · Fixed by #1322
Closed

Update package metadata for all packages #1259

srikanthccv opened this issue Sep 1, 2022 · 14 comments · Fixed by #1322

Comments

@srikanthccv
Copy link
Member

Following this open-telemetry/opentelemetry-python#2867, remove setup.py, setup.cfg in favor of pyproject.toml to specify metadata for all packages. This issue can be used as a overall tracking issue. Feel free to open up individual issues per package and link them to this one.

This was referenced Sep 2, 2022
@ofek
Copy link
Contributor

ofek commented Sep 2, 2022

Progress:

@ofek
Copy link
Contributor

ofek commented Sep 5, 2022

@srikanthccv @ocelotl Everything is passing except for py*-test-instrumentation-grpc which is failing on all new PRs

@srikanthccv
Copy link
Member Author

Have seen this happen intermittently. I will look into it.

@srikanthccv
Copy link
Member Author

I think the generate workflow is failing for the same reason described here open-telemetry/opentelemetry-python#2920

@ofek
Copy link
Contributor

ofek commented Sep 7, 2022

#1318

@ofek
Copy link
Contributor

ofek commented Sep 8, 2022

all pass now

@srikanthccv
Copy link
Member Author

srikanthccv commented Sep 8, 2022

@srikanthccv
Copy link
Member Author

@ofek this script file needs to be updated to use the global config. We can't merge the PR unless that workflow also passes.

@ofek ofek mentioned this issue Sep 9, 2022
@ofek
Copy link
Contributor

ofek commented Sep 9, 2022

#1322

@ofek
Copy link
Contributor

ofek commented Sep 10, 2022

oops need to reopen

@srikanthccv srikanthccv reopened this Sep 10, 2022
@ofek
Copy link
Contributor

ofek commented Sep 13, 2022

complete 😄

@srikanthccv
Copy link
Member Author

I will close this after this gets merged #1329

@ofek
Copy link
Contributor

ofek commented Sep 14, 2022

@ofek
Copy link
Contributor

ofek commented Sep 14, 2022

take care!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants