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

Add pre-commit jobs to lint ReST files #293

Merged
merged 5 commits into from
Jan 24, 2025
Merged

Conversation

timwoj
Copy link
Member

@timwoj timwoj commented Jan 23, 2025

This fixes a bunch of ReST style issues, mostly for consistency across all of the docs. This includes regenerated changes from zeek/zeek#4193.

@timwoj timwoj force-pushed the topic/timw/rst-linting branch from 464d0d1 to 0dcbdee Compare January 23, 2025 21:10
Copy link
Contributor

@awelzel awelzel left a comment

Choose a reason for hiding this comment

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

Thanks for making this pass! Just some additional nits/ideas.

devel/spicy/reference.rst Outdated Show resolved Hide resolved
frameworks/notice.rst Outdated Show resolved Hide resolved
scripting/javascript.rst Outdated Show resolved Hide resolved
scripting/javascript.rst Outdated Show resolved Hide resolved
scripting/javascript.rst Outdated Show resolved Hide resolved
scripting/optimization.rst Outdated Show resolved Hide resolved
@timwoj timwoj force-pushed the topic/timw/rst-linting branch from 396dada to 459da33 Compare January 24, 2025 18:21
@timwoj timwoj force-pushed the topic/timw/rst-linting branch from 459da33 to eb19783 Compare January 24, 2025 18:30
@timwoj timwoj force-pushed the topic/timw/rst-linting branch from eb19783 to b36015c Compare January 24, 2025 18:33
@timwoj timwoj merged commit c78d4a2 into master Jan 24, 2025
11 checks passed
@timwoj timwoj deleted the topic/timw/rst-linting branch January 24, 2025 18:33
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.

2 participants