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.

Sign up for GitHub

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

Jump to bottom

Travis: Update to sphinx 1.8.3 #396

Closed
wants to merge 1 commit into from
Closed

Travis: Update to sphinx 1.8.3 #396

wants to merge 1 commit into from

Conversation

t-b
Copy link
Contributor

@t-b t-b commented Sep 27, 2018

Closes #394.

This is to independently verify if it works or not.

@t-b t-b changed the title Travis: Update to sphinx 1.8.1 Travis: Update to sphinx 1.8.2 Nov 13, 2018
@t-b t-b force-pushed the update-sphinx-to-1.8.1-on-travis branch from 3cad772 to 285e760 Compare November 13, 2018 09:33
@t-b t-b force-pushed the update-sphinx-to-1.8.1-on-travis branch from 285e760 to 6f0ff34 Compare January 7, 2019 17:43
@t-b t-b changed the title Travis: Update to sphinx 1.8.2 Travis: Update to sphinx 1.8.3 Jan 7, 2019
@vermeeren
Copy link
Collaborator

#410 has just been merged, which should fix tests for 1.8.x series.

I am not sure how this impacts #394, reading that issue it currently seems to be half-resolved as there are still a lot of warnings that need to be fixed when breathe is refactored/rewritten. What do you think?

@vermeeren vermeeren added the test Unit tests, acceptance tests label Jan 23, 2019
@t-b
Copy link
Contributor Author

t-b commented Jan 23, 2019

I'm okay with closing #394 as fixing all warnings is a nice longterm goal but for now I'm fine with the fact that it works with the latest 1.8.x.

@t-b t-b closed this Jan 23, 2019
@vermeeren vermeeren self-assigned this Jan 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@vermeeren vermeeren

Labels
test Unit tests, acceptance tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@t-b @vermeeren