Skip to content
This repository has been archived by the owner on May 30, 2022. It is now read-only.

Added poetry and lock pyparsing to ^2.4.7. #86

Closed
wants to merge 1 commit into from
Closed

Conversation

AndreasBackx
Copy link
Contributor

@AndreasBackx AndreasBackx commented Dec 1, 2021

I think it's a good idea to revisit the discussion from #20. The tools have matured and considering the current state of Nubia, this would at least fix the problems we are having. Using poetry is considerably simpler and distutils is deprecated.

The following changes are also included:

  • Version change to 0.3.0. With the move to Poetry, I think a minor version upgrade is warranted.
  • All dependencies have been set to the latest version and pyparsing to ^2.4.7 as 3.x is not supported right now.

Resolves #85.

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Dec 1, 2021
@AndreasBackx AndreasBackx marked this pull request as ready for review December 1, 2021 14:35
@AndreasBackx
Copy link
Contributor Author

After consideration. Might be best to not change the version here and bump it to 0.3.0 together with the async changes from #81.

@facebook-github-bot
Copy link
Contributor

@AndreasBackx has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@AhmedSoliman AhmedSoliman self-requested a review December 6, 2021 11:44
vinspam added a commit to vinspam/python-nubia that referenced this pull request Oct 30, 2024
Summary:
I think it's a good idea to revisit the discussion from facebookarchive/python-nubia#20. The tools have matured and considering the current state of Nubia, this would at least fix the problems we are having. Using poetry is considerably simpler and [distutils is deprecated](https://www.python.org/dev/peps/pep-0632/).

The following changes are also included:
* All dependencies have been set to the latest version and pyparsing to `^2.4.7` as `3.x` is not supported right now.

Resolves facebookarchive/python-nubia#85.

Pull Request resolved: facebookarchive/python-nubia#86

Reviewed By: AhmedSoliman

Differential Revision: D32879106

Pulled By: AndreasBackx

fbshipit-source-id: e07de2b1eaba3a9e82a5d12493a7079297e42727
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed Do not delete this pull request or issue due to inactivity.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setup.py needs to pin pyparsing 2.4.7
2 participants