Skip to content

Fix [Sphinx] [Automation] Issue -> PR conversion of code blocks | Closes #211#227

Merged
plaindocs merged 6 commits into
Safety-Critical-Rust-Consortium:mainfrom
MarcosBorgesPhD:pr-211
Dec 3, 2025
Merged

Fix [Sphinx] [Automation] Issue -> PR conversion of code blocks | Closes #211#227
plaindocs merged 6 commits into
Safety-Critical-Rust-Consortium:mainfrom
MarcosBorgesPhD:pr-211

Conversation

@MarcosBorgesPhD

@MarcosBorgesPhD MarcosBorgesPhD commented Nov 20, 2025

Copy link
Copy Markdown
Contributor

Fix Issue -> PR conversion of code blocks.

Details

  • Dedent before adding indentation
  • Add correct indentation

Notes

This is PR 2/3 in a series improving Issue -> PR conversion:

  1. PR 1: Fix hyperlinks [Sphinx] [Automation] Make sure Issue -> PR conversion of hyperlinks is correct #221
  2. PR 2: Fix code block conversion [Sphinx] [Automation] Make sure Issue -> PR conversion of code blocks is correct #211
  3. PR 3: Fix table conversion [Sphinx] [Automation] Make sure Issue -> PR conversion of tables is correct #210

Closes #221, #211

@netlify

netlify Bot commented Nov 20, 2025

Copy link
Copy Markdown

Deploy Preview for scrc-coding-guidelines ready!

Name Link
🔨 Latest commit b432353
🔍 Latest deploy log https://app.netlify.com/projects/scrc-coding-guidelines/deploys/692f5e5d4f6fc40008faecc2
😎 Deploy Preview https://deploy-preview-227--scrc-coding-guidelines.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@PLeVasseur

Copy link
Copy Markdown
Collaborator

Thanks for this @MarcosBorgesPhD! I'll try to make time this week to review

- Dedent before adding indentation
- Add correct indentation
- Convert inline-code links like [`text`](url) to standard [text](url)
- Correct mixed bold/code formatting:
    - **`code`** → `code`
    - `**code**` → `code`

This prevents Markdown parser errors in issue bodies and ensures consistent formatting.
Resolve ruff lint error I001 that was preventing the build.
@MarcosBorgesPhD

Copy link
Copy Markdown
Contributor Author

@PLeVasseur Apologies for the amount of notifications.

  1. This PR has been fully rebased onto upstream/main, which should fix the CI failures.

  2. The commit contains only the files relevant to this issue.

@plaindocs plaindocs left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good to me.

@plaindocs
plaindocs added this pull request to the merge queue Dec 3, 2025
Merged via the queue into Safety-Critical-Rust-Consortium:main with commit 806bbfe Dec 3, 2025
7 checks passed
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.

[Sphinx] [Automation] Make sure Issue -> PR conversion of hyperlinks is correct

3 participants