Skip to content

Conversation

@MarcosBorgesPhD
Copy link
Contributor

@MarcosBorgesPhD MarcosBorgesPhD commented Nov 20, 2025

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
Copy link

netlify bot commented Nov 20, 2025

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
Collaborator

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

- Dedent before adding indentation
- Add correct indentation
@MarcosBorgesPhD MarcosBorgesPhD force-pushed the pr-211 branch 2 times, most recently from 55acc5c to 56aff1b Compare December 2, 2025 21:45
- 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
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.

Copy link
Collaborator

@plaindocs plaindocs left a comment

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 rustfoundation: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