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

Switch RTD build to fail on gallery example failure #3619

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

zm711
Copy link
Collaborator

@zm711 zm711 commented Jan 14, 2025

I included our options in a comment to make it easier to edit in the future.

Fixes #3603

@zm711 zm711 added the documentation Improvements or additions to documentation label Jan 14, 2025
@h-mayorquin
Copy link
Collaborator

This will require a change on the settings to add the passing of this check as a requirement for the pull request to be merged, right?

@zm711
Copy link
Collaborator Author

zm711 commented Jan 14, 2025

I don't think we wanted it required. More just the shame of the red x rather than the green check. So that we are aware the PR caused problems.

@alejoe91 alejoe91 merged commit 01d1479 into SpikeInterface:main Jan 14, 2025
15 checks passed
@h-mayorquin
Copy link
Collaborator

Ah, got it wrong. Thanks, guys.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Should we fail doc build on broken example moving forward?
3 participants