Skip to content

Conversation

@odow
Copy link
Contributor

@odow odow commented Apr 17, 2023

@odow odow mentioned this pull request Apr 17, 2023
7 tasks
@odow
Copy link
Contributor Author

odow commented Apr 17, 2023

Hmm. This still failed locally. I need to take a deeper look

@odow odow changed the title Fix missing import in tests Remove test/Project.toml Apr 17, 2023
@odow
Copy link
Contributor Author

odow commented Apr 17, 2023

If you include an extras section in Project.toml, then it's just confusing to have test/Project.toml.

# TODO Should be fixed in MOI master
"test_model_LowerBoundAlreadySet",
"test_model_UpperBoundAlreadySet",
"test_linear_open_intervals",
Copy link
Member

@blegat blegat Apr 17, 2023

Choose a reason for hiding this comment

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

It should be excluded in the else clause. But it should work with MOI master now thanks to #82

@odow
Copy link
Contributor Author

odow commented Apr 17, 2023

Starting again because you merged #83

@odow odow closed this Apr 17, 2023
@odow odow deleted the patch-1 branch April 17, 2023 21:40
@odow odow mentioned this pull request Apr 17, 2023
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