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

Fix ci win #526

Closed
wants to merge 8 commits into from
Closed

Fix ci win #526

wants to merge 8 commits into from

Conversation

k-dominik
Copy link
Collaborator

No description provided.

hmaarrfk and others added 8 commits October 23, 2022 21:39
Co-authored-by: Dominik Kutra <k-dominik@users.noreply.github.com>
build will fail otherwise

would probably better to fix such that also vigranumpy tests are not build
with BUILD_TESTS=OFF
this will indirectly force 14.1 toolset when building, otherwise there is a
standing issue with 14.2 (VS2019).

xref: ref: ukoethe#525
@k-dominik
Copy link
Collaborator Author

k-dominik commented Dec 5, 2022

Hey @hmaarrfk,

this PR seems to fix the windows ci - do you want to squash the content and add it to #514 ?

@hmaarrfk
Copy link
Collaborator

hmaarrfk commented Dec 5, 2022

Yes. Feel free to push the necessary changes to 514. If possible the git history would "make sense"

@hmaarrfk hmaarrfk closed this Feb 19, 2023
@hmaarrfk hmaarrfk reopened this Feb 19, 2023
@hmaarrfk
Copy link
Collaborator

I cherry picked and folded into #514 lets see how far we go now

@hmaarrfk hmaarrfk mentioned this pull request Feb 21, 2023
19 tasks
@k-dominik
Copy link
Collaborator Author

k-dominik commented Mar 7, 2023

closed - fixed already by @hmaarrfk in #514

@k-dominik k-dominik closed this Mar 7, 2023
@k-dominik k-dominik deleted the fix_ci_win branch March 7, 2023 12:56
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.

2 participants