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

Add mypy dependency to build requirements in meta.yaml #15

Closed
wants to merge 14 commits into from

Conversation

alopezrivera
Copy link
Collaborator

No description provided.

@alopezrivera alopezrivera changed the base branch from master to develop June 25, 2024 17:03
@geoffreygarrett geoffreygarrett self-assigned this Jun 28, 2024
@geoffreygarrett geoffreygarrett added the bug Something isn't working label Jun 28, 2024
recipe/build.sh Outdated Show resolved Hide resolved
recipe/meta.yaml Outdated Show resolved Hide resolved
@geoffreygarrett

This comment was marked as resolved.

recipe/meta.yaml Outdated Show resolved Hide resolved
@DominicDirkx
Copy link
Member

Hi all, this issue started with the release of v0.8.0.dev9, which caused a whole bunch of issues in the tudatpy build that seem difficult to resolve. I've rolled back the release, and have re-opened the pull request:

tudat-team/tudatpy#154

before merging this, I propose we fix the Azure build of the associated changes, by hard-coding the commit in the recipe.yaml, rather than assigning a version. That way, we can merge other tudatpy features, while we resolve this one on the side. I'll patch up the develop branch of the feedstock.

@geoffreygarrett
Copy link
Member

geoffreygarrett commented Jul 11, 2024

AFAIK https://github.com/sizmailov/pybind11-stubgen would be the way to go if mypy was intended for stub generation for code completion etc. I've used it successfully in the past. @alopezrivera

@geoffreygarrett
Copy link
Member

I'll close this for now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants