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

Subdivide tests and rewrite try/catch/continue #25

Merged
merged 2 commits into from Aug 14, 2016
Merged

Subdivide tests and rewrite try/catch/continue #25

merged 2 commits into from Aug 14, 2016

Conversation

swt30
Copy link
Contributor

@swt30 swt30 commented Aug 9, 2016

Continue statements in tests cause problems (JuliaLang/julia#17908) so I rewrote those, split the tests into individual test sets, and added a missing import. The tests now work on julia 0.4, but 0.5 still has issues.

Continue statements in tests cause problems (JuliaLang/julia#17908)
@coveralls
Copy link

coveralls commented Aug 9, 2016

Coverage Status

Changes Unknown when pulling 7a04d03 on swt30:fix-tests into * on JuliaGeometry:master*.

@swt30
Copy link
Contributor Author

swt30 commented Aug 12, 2016

Tests should now work on julia 0.5 once a new GeometricalPredicates is tagged.

@coveralls
Copy link

coveralls commented Aug 12, 2016

Coverage Status

Changes Unknown when pulling b98c79c on swt30:fix-tests into * on JuliaGeometry:master*.

@swt30 swt30 changed the title Subdivide tests and rewrite try/catch/continue Subdivide tests and rewrite try/catch/continue; closes #24 Aug 12, 2016
@swt30 swt30 changed the title Subdivide tests and rewrite try/catch/continue; closes #24 Subdivide tests and rewrite try/catch/continue Aug 12, 2016
@coveralls
Copy link

coveralls commented Aug 12, 2016

Coverage Status

Changes Unknown when pulling 0c541f6 on swt30:fix-tests into * on JuliaGeometry:master*.

@skariel skariel merged commit 45d8110 into JuliaGeometry:master Aug 14, 2016
@skariel
Copy link
Contributor

skariel commented Aug 14, 2016

Merged, thanks

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.

None yet

3 participants