[stable33] backport: Add sphinx-lint CI + fix all RST violations (PR#14769 + PR#14771) to stable33#14814
Merged
Merged
Conversation
- Add sphinx-lint==1.0.2 to requirements.txt - Create .github/workflows/sphinx-lint.yml (disabling trailing-whitespace and horizontal-tab which have thousands of pre-existing violations) - Fix 38 pre-existing RST violations across all three manuals: - 27 files missing final newline - Malformed hyperlinks (missing space, missing underscore) - Unbalanced inline literal delimiters - Role markup errors (missing backticks, missing colons) - Dangling hyphen in URL (converted to inline hyperlink) Agent-Logs-Url: https://github.com/nextcloud/documentation/sessions/13a4f646-2926-4695-8df9-87521fbb2383 Co-authored-by: skjnldsv <14975046+skjnldsv@users.noreply.github.com>
Agent-Logs-Url: https://github.com/nextcloud/documentation/sessions/13a4f646-2926-4695-8df9-87521fbb2383 Co-authored-by: skjnldsv <14975046+skjnldsv@users.noreply.github.com>
…t dual-install intent Agent-Logs-Url: https://github.com/nextcloud/documentation/sessions/c8666b05-c4aa-478f-a667-a8619d24faae Co-authored-by: skjnldsv <14975046+skjnldsv@users.noreply.github.com>
Agent-Logs-Url: https://github.com/nextcloud/documentation/sessions/7e98a96f-e1cf-4659-9ee7-928e22178f34 Co-authored-by: skjnldsv <14975046+skjnldsv@users.noreply.github.com>
Removed specific linting rules to simplify sphinx-lint command. Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com> Co-authored-by: skjnldsv <14975046+skjnldsv@users.noreply.github.com>
…iolations Agent-Logs-Url: https://github.com/nextcloud/documentation/sessions/229670e0-d357-4027-9641-98496c8d3111 Co-authored-by: skjnldsv <14975046+skjnldsv@users.noreply.github.com>
Agent-Logs-Url: https://github.com/nextcloud/documentation/sessions/020edc0e-0c24-4574-8feb-6bc06b0e14fc Co-authored-by: skjnldsv <14975046+skjnldsv@users.noreply.github.com>
Co-authored-by: skjnldsv <14975046+skjnldsv@users.noreply.github.com>
Copilot created this pull request from a session on behalf of
skjnldsv
May 12, 2026 07:57
View session
…ty vulnerabilities Agent-Logs-Url: https://github.com/nextcloud/documentation/sessions/88e127f7-7717-426f-8e3f-aa500cf40898 Co-authored-by: skjnldsv <14975046+skjnldsv@users.noreply.github.com>
skjnldsv
force-pushed
the
copilot/backport-14769-and-add-14771
branch
from
8616781 to
3a92747
Compare
skjnldsv
requested review from
Antreesy,
ChristophWurst,
DorraJaouad,
GVodyanov,
GretaD,
SebastianKrupinski,
hamza221,
kesselb and
nickvergessen
as code owners
skjnldsv
changed the title
Antreesy
approved these changes
May 12, 2026
Contributor
Antreesy
left a comment
Antreesy
left a comment
There was a problem hiding this comment.
Looks good, cherry-pick seems to be complete
Contributor
Member
|
@copilot resolve the merge conflicts in this pull request |
…14769-and-add-14771 Co-authored-by: skjnldsv <14975046+skjnldsv@users.noreply.github.com>
Contributor
Author
Resolved in commit |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Backport of #14769 and #14771 to
stable33, plus security dependency bumps and merge conflict resolution.☑️ Resolves
trailing-whitespaceandhorizontal-tab#14771 (Enabletrailing-whitespaceandhorizontal-tabsphinx-lint checks)🖼️ Screenshots
No visual changes (CI/lint and dependency updates only).
From PR #14769:
.github/workflows/sphinx-lint.yml— lightweight CI job runningsphinx-linton all three manuals.github/problem-matchers/sphinx-lint.json— inline PR annotations for sphinx-lint findingsrequirements.txt: addedsphinx-lint==1.0.2From PR #14771:
--disable trailing-whitespace --disable horizontal-tabflags → full lint enforcementSecurity dependency bumps:
Pillow==12.1.0→12.2.0(fixes OOB write with invalid PSD tile extents, FITS GZIP decompression bomb, OOB write loading PSD images)urllib3==2.6.3→2.7.0(fixes decompression-bomb safeguard bypass, sensitive headers forwarded across origins in proxied redirects)Conflict resolution:
config_sample_php_parameters.rstregeneration) with whitespace fixes re-appliedsphinx-lint admin_manual developer_manual user_manual→No problems found✅