-
-
Notifications
You must be signed in to change notification settings - Fork 7.1k
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
fix: Allow equals sign in sequenceDiagram labels #6332
base: develop
Are you sure you want to change the base?
fix: Allow equals sign in sequenceDiagram labels #6332
Conversation
🦋 Changeset detectedLatest commit: 6cc1926 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
commit: |
✅ Deploy Preview for mermaid-js ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #6332 +/- ##
=======================================
Coverage 3.88% 3.88%
=======================================
Files 398 398
Lines 42044 42044
Branches 638 638
=======================================
Hits 1635 1635
Misses 40409 40409
Flags with carried forward coverage won't be shown. Click here to find out more. |
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
@sidharthv96 @ashishjain0512 Could either of you please take a look? Looks like a straightforward fix. This is one of the issues that will help us upgrade Mermaid from v10.x to v11.x easily in GitLab: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/171452 |
b773fd5
to
6cc1926
Compare
@sidharthv96 @ashishjain0512 I've added a changeset, rebased the change on top of latest Ideally, this PR and #6345 could be merged before the next release. I believe these two PRs would be sufficient to allow GitLab to more easily adopt the 11.x series. |
📑 Summary
Resolves the issue where equals sign was accidentally disallowed in sequence diagram participants.
Resolves #6171
📏 Design Decisions
Commit e852156 unintentionally changed an escaped hyphen in a regex character class to an escaped left-angle-bracket followed by an un-escaped hyphen. Rather than disallowing <, -, and >, this change inadvertently disallowed characters
<
through>
. Nestled in between<
and>
in the ASCII characters is=
.📋 Tasks
Make sure you
MERMAID_RELEASE_VERSION
is used for all new features.pnpm changeset
and following the prompts. Changesets that add features should beminor
and those that fix bugs should bepatch
. Please prefix changeset messages withfeat:
,fix:
, orchore:
.