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

Remove deprecated pkg_resources #1119

Merged
merged 2 commits into from
Nov 17, 2023
Merged

Conversation

NeilGirdhar
Copy link
Contributor

Tests

  • I added corresponding tests for bug fixes and new features. If possible, the tests fail without the changes
  • All new and existing tests are passing ($ tox -e py38; replace 38 by your Python version if necessary)

Documentation

  • I have updated the relevant documentation related to my changes

Quality

  • I have read the CONTRIBUTING doc
  • My commits messages follow this format
  • My code follows the style guidelines ($ tox -e lint)

Further comments

Please include any additional information or comment that you feel will be helpful to the review of this pull request.

@bouthilx
Copy link
Member

Seems like there was an issue with the import here: https://github.com/Epistimio/orion/actions/runs/6471640548/job/17570730647?pr=1119#step:5:92

@NeilGirdhar
Copy link
Contributor Author

Seems like there was an issue with the import here:

Fixed.

@bouthilx
Copy link
Member

Thanks! Now there seems to be an issue with the entry point object: https://github.com/Epistimio/orion/actions/runs/6486272775/job/17614328726?pr=1119#step:5:745

@NeilGirdhar
Copy link
Contributor Author

Thanks! Now there seems to be an issue with the entry point object:

Fixed. The Pylint error is because the Pylint you're running on 3.9 is too old IMO. Blocking that error is goign to be confusing since it's not an error in newer Pythons.

@NeilGirdhar
Copy link
Contributor Author

@bouthilx Would you like me to do anything else for this pull request?

Copy link
Member

@bouthilx bouthilx left a comment

Choose a reason for hiding this comment

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

I believe your point about pylint is valid. Let's wait for #1121 to be merged before just to make sure all tests are passing here.

Thanks!

@bouthilx bouthilx merged commit 1b20511 into Epistimio:develop Nov 17, 2023
20 of 25 checks passed
@NeilGirdhar NeilGirdhar deleted the pkgre branch November 17, 2023 21:44
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.

2 participants