Skip to content
This repository has been archived by the owner on Mar 15, 2024. It is now read-only.

Make markdown refs compatible with prettier #181

Merged
merged 2 commits into from
Jan 21, 2022

Conversation

ssbarnea
Copy link
Member

@ssbarnea ssbarnea commented Jan 20, 2022

Ensures that all our references keys are using lowercase in order to workaround known prettier bug.

This is also sorting another issue with editing markdown in vscode itself, which seems to not recognize or highlight the single [foo] blocks but works well on full ones like [Foo][x] and even inter-linking them.

Related: prettier/prettier#3835
Required-By: #177

Ensures that all our references keys are using lowercase in order
to workaround known prettier bug.

Related: prettier/prettier#3835
Required-By: ansible#177
@ssbarnea ssbarnea marked this pull request as ready for review January 20, 2022 12:46
@ssbarnea ssbarnea requested a review from a team as a code owner January 20, 2022 12:46
Copy link
Contributor

@tomaciazek tomaciazek left a comment

Choose a reason for hiding this comment

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

LGTM!

@ssbarnea ssbarnea merged commit 4c65138 into ansible:main Jan 21, 2022
@ssbarnea ssbarnea deleted the fix/prettier-refs branch January 21, 2022 09:03

[CoC]:
[coc]:
Copy link
Member

Choose a reason for hiding this comment

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

I asked you not to do the lowercasing in another PR 🤦‍♂️ Why did you sneak it in anyway?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants