Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix CI #155

Closed
wants to merge 7 commits into from
Closed

Conversation

adrien-berchet
Copy link
Contributor

@adrien-berchet adrien-berchet commented Apr 4, 2021

Fix the Numpy versions and use Anaconda to make the CI pass.

@adrien-berchet adrien-berchet marked this pull request as ready for review June 22, 2021 08:52
@adrien-berchet
Copy link
Contributor Author

Hello
Here is an attempt to fix the CI so it is possible to merge #152 (I would really need it) and make a new release.

@joesolly
Copy link
Contributor

What do you think about this solution for the builds? #158

@adrien-berchet
Copy link
Contributor Author

I prefer it as it is much simpler, I just didn't want to change the requirements but if it's ok for you it's ok for me :)

@joesolly
Copy link
Contributor

I prefer it as it is much simpler, I just didn't want to change the requirements but if it's ok for you it's ok for me :)

In my opinion, it's totally fine. It is only for an extra requirement install, it does not restrict someone from installing a different version of Numpy on their own.

@adrien-berchet
Copy link
Contributor Author

I agree, let's take your version.

@adrien-berchet adrien-berchet deleted the fix_ci branch July 22, 2021 07:56
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants