Skip to content

rst-lint -> rstcheck#23

Open
peterjc wants to merge 5 commits intomasterfrom
rstcheck
Open

rst-lint -> rstcheck#23
peterjc wants to merge 5 commits intomasterfrom
rstcheck

Conversation

@peterjc
Copy link
Copy Markdown
Owner

@peterjc peterjc commented Mar 9, 2020

Work in progress exploring using rstcheck which adds support for RST blocks in other languages like Python, Bash or JSON. Would address #22

peterjc added 2 commits March 8, 2020 15:24
@peterjc
Try restructuredtext_lint -> rstcheck
c576839 
Idea here is rstcheck will also validate python
blocks etc, as well as the reStructuredText via
docutils. That would need new error codes:

$ flake8 --select RST some_python.py
some_python.py:8:1: RST999 Unexpected prefix: '(python) unexpected EOF while parsing'

See GitHub issue #22, using the example there:

def some():
    """
    ====
    Test
    ====

    .. code:: python

        print(

    End.
    """
    pass
@peterjc
Map changed msg for RST303 and RST304
ab417e9 
A bit of a hack as change from rst-lint to rstcheck
was not quite transparent.
@peterjc
Copy link
Copy Markdown
Owner Author

peterjc commented Mar 9, 2020

Looks like can call rstcheck. ignore_directives_and_roles(...) to tell it additional directives and roles to ignore in advance

@peterjc
Copy link
Copy Markdown
Owner Author

peterjc commented Apr 15, 2021

Currently no easy way to stop rstlint automatically ignoring Sphinx directives and roles, which would ideally be configurable.

@peterjc
Copy link
Copy Markdown
Owner Author

peterjc commented Apr 16, 2022

Cross reference rstcheck/rstcheck#65 (comment) - with rstcheck under active maintenance again, this is worth revisiting.

@peterjc peterjc mentioned this pull request Apr 16, 2022
Closed
@Cielquan
Copy link
Copy Markdown

Cielquan commented Apr 18, 2022

For the future: When this PR is merged we could add a link to rstcheck pointing to this repo in a Usage with flake8 section or so.

@peterjc
Copy link
Copy Markdown
Owner Author

peterjc commented Apr 21, 2022

See rstcheck/rstcheck#95 - may be best waiting for the API to settle before refreshing this.

@Cielquan
Copy link
Copy Markdown

Cielquan commented May 13, 2022

I just released v6.0.0a1 of rstcheck. If you would like to take a look.

The roadmap until the full release is here: rstcheck/rstcheck#104

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@peterjc @Cielquan