Skip to content

Conversation

rinOfTheStars
Copy link
Contributor

@rinOfTheStars rinOfTheStars commented Aug 24, 2025

From what I understand, the auto-formatting/prettifier plugin used by Cursorless really didn't like how CONTRIBUTING.MDX needed to be formatted to have the separation between vscode and vscodium specific commands show properly, and as a result broke a bunch of the code blocks, transforming them from

echo hello yes this is dog

to
bash echo hello yes this is dog
though ofc the commands were actual commands and not an echo example. I forget if any other formatting was broken, it has been a while since I actually finished the work on this and I just forgot to make a PR for it, sorry (been busy with getting ready for the new semester, health nonsense, and non-CS hobbies). This PR fixes the aforementioned issue and excludes CONTRIBUTING.MDX from being processed by the auto-formatting plugin, allowing it to contain all of the weird extended markdown jank the plugin seems to really not like. If there's a more elegant way to do this (changing the rules the plugin follows?) please lmk and commit it if you have the time, this is very much not my usual wheelhouse.

@rinOfTheStars rinOfTheStars requested a review from a team as a code owner August 24, 2025 18:56
@AndreasArvidsson AndreasArvidsson added this pull request to the merge queue Aug 24, 2025
Merged via the queue into cursorless-dev:main with commit cab31d1 Aug 24, 2025
15 checks passed
Copy link
Member

@auscompgeek auscompgeek left a comment

Choose a reason for hiding this comment

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

Have we reported the bug upstream yet?

<TabItem value="vscodium" label="VSCodium">
`bash codium --profile=cursorlessDevelopment --install-extension
bundle.vsix`
```bash
Copy link
Member

Choose a reason for hiding this comment

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

I suspect if we put blank lines around the code blocks prettier will have a better time

Copy link
Member

Choose a reason for hiding this comment

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

Thank you! That worked.

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.

3 participants