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

Ignore pyparsing type hints #1802

Conversation

aucampia
Copy link
Member

Summary of changes

The type hints from pyparsing 3.0.8 does not work for rdflib:
pyparsing/pyparsing#385

Once the type hints for pyparsing is in a better state we can enable
them again.

Checklist

  • Checked that there aren't other open pull requests for
    the same change.
  • Checked that all tests and type checking passes.
  • Considered granting push permissions to the PR branch,
    so maintainers can fix minor issues and keep your PR up to date.

@aucampia aucampia force-pushed the iwana-20220410T2049-ignore_pyparsing_type_hints branch from 93d5777 to 603b7be Compare April 10, 2022 18:54
@aucampia aucampia mentioned this pull request Apr 10, 2022
3 tasks
The type hints from pyparsing 3.0.8 does not work for rdflib:
pyparsing/pyparsing#385

Once the type hints for pyparsing is in a better state we can enable
them again.
@aucampia aucampia force-pushed the iwana-20220410T2049-ignore_pyparsing_type_hints branch from 603b7be to fe589d0 Compare April 10, 2022 20:17
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

I'm good with this.

@aucampia
Copy link
Member Author

aucampia commented Apr 10, 2022

Will merge tomorrow, everything else will have to be rebased or have master branch merged into it to prevent this error from affecting us, I will look at pyparsing this summer if it is still not fixed.

EDIT: actually just PRs that are actually failing will have to be rebased, everything else is actually okay.

@aucampia aucampia merged commit 7ed86ff into RDFLib:master Apr 11, 2022
@aucampia
Copy link
Member Author

Just a note, this is fixed in upstream, will just wait for a new release.

@aucampia aucampia deleted the iwana-20220410T2049-ignore_pyparsing_type_hints branch April 14, 2022 09:29
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.

None yet

1 participant