Skip to content

lint: enable sphinx-lint for Sphinx extension#1266

Merged
larsoner merged 6 commits intosphinx-gallery:masterfrom
Borda:lint/sphinx
Mar 11, 2024
Merged

lint: enable sphinx-lint for Sphinx extension#1266
larsoner merged 6 commits intosphinx-gallery:masterfrom
Borda:lint/sphinx

Conversation

@Borda
Copy link
Copy Markdown
Contributor

@Borda Borda commented Mar 8, 2024

I think it is nature to have validated Sphinx extension with sphinx-lint tool 🐿️

Copy link
Copy Markdown
Contributor

@lucyleeow lucyleeow left a comment

Choose a reason for hiding this comment

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

Thank you! The trailing whitespace really annoyed me. Just a question.

@Borda Borda requested a review from larsoner March 11, 2024 12:24
@Borda
Copy link
Copy Markdown
Contributor Author

Borda commented Mar 11, 2024

@larsoner @lucyleeow seems I have resolved all/last issues... mind checking 🦩

@larsoner
Copy link
Copy Markdown
Contributor

Thanks @Borda !

@larsoner larsoner merged commit 50fdf82 into sphinx-gallery:master Mar 11, 2024
@Borda Borda deleted the lint/sphinx branch March 11, 2024 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants