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

Added typing_extensions to dependencies. #112

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

coltonbh
Copy link
Contributor

Description

Fixes #111

Changelog description

Added typing_extensions to dependencies.

Status

Good to go!

Copy link
Member

@awvwgk awvwgk left a comment

Choose a reason for hiding this comment

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

Good catch, thanks.

@coltonbh
Copy link
Contributor Author

coltonbh commented Jul 5, 2024

Hi @awvwgk possible to merge and fix? Thx :)

@awvwgk
Copy link
Member

awvwgk commented Jul 5, 2024

Thanks for the patch. However since the CI is not passing it is hard to merge

@coltonbh
Copy link
Contributor Author

coltonbh commented Jul 5, 2024

Indeed! However the CI is failing for reasons unrelated to the patch.

Is CI passing on master branch? I.e., have you fixed these build issues already? If so I'll rebase and push.

hppritcha and others added 2 commits July 5, 2024 11:35
At least with meson 1.3.2 one has to set pure to false
for find installation or else build fails.

Signed-off-by: Howard Pritchard <howardp@lanl.gov>
…s check for third party dependency package. This dependency is correctly specified in pyproject.toml. (grimme-lab#109)
@coltonbh
Copy link
Contributor Author

coltonbh commented Jul 5, 2024

Rebased off latest changes from main branch that appear to address some build issues...

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.

typing_extentions missing from dependencies.
3 participants