Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

All the recent changes, now against master 🤦‍♂️ #163

Merged
merged 5 commits into from
Dec 8, 2024

Conversation

movermeyer
Copy link
Collaborator

What are you trying to accomplish?

I accidentally merged these all into the movermeyer/python_313_support branch instead of master:

What approach did you choose and why?

Opened a PR with these commits to get them into master.

What should reviewers focus on?

See the individual PRs for descriptions of the changes.

The impact of these changes

See the individual PRs for descriptions of the changes.

Testing

...

@movermeyer movermeyer changed the title Movermeyer/python 313 support All the recent changes, now against master 🤦‍♂️ Dec 5, 2024
Previously, one could include backticks in their node labels. However, Mermaid v10 introduced "Markdown formatting" (which despite its name is not Markdown, but just Markdown-style bold and italics) which uses the backticks as part of its delimiters. This broke these labels.

I've changed the labels to no longer use the backticks. 🤷
@movermeyer movermeyer force-pushed the movermeyer/python_313_support branch from 3518c46 to c465732 Compare December 5, 2024 11:48
@movermeyer movermeyer marked this pull request as ready for review December 5, 2024 11:50
@movermeyer movermeyer requested a review from thomasst December 5, 2024 11:52
@movermeyer movermeyer merged commit d6d021f into master Dec 8, 2024
18 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.

2 participants