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

DOCS: update announcement bar #5193

Merged
merged 2 commits into from Jan 12, 2024
Merged

DOCS: update announcement bar #5193

merged 2 commits into from Jan 12, 2024

Conversation

huynhicode
Copy link
Member

📑 Summary

This PR contains updates to the Announcement bar.

Resolves # n/a

📏 Design Decisions

Design decisions made by the Mermaid Chart team.

📋 Tasks

Make sure you

Copy link

netlify bot commented Jan 12, 2024

Deploy Preview for mermaid-js failed.

Name Link
🔨 Latest commit d6d465d
🔍 Latest deploy log https://app.netlify.com/sites/mermaid-js/deploys/65a160cfc98a9d0008809546

Copy link

codecov bot commented Jan 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (749db74) 76.57% compared to head (d6d465d) 76.60%.
Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5193      +/-   ##
==========================================
+ Coverage   76.57%   76.60%   +0.03%     
==========================================
  Files         164      164              
  Lines       13820    13820              
  Branches      693      693              
==========================================
+ Hits        10582    10587       +5     
+ Misses       3080     3077       -3     
+ Partials      158      156       -2     
Flag Coverage Δ
e2e 81.77% <ø> (+0.02%) ⬆️
unit 42.94% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 12 files with indirect coverage changes

@huynhicode huynhicode changed the base branch from develop to master January 12, 2024 06:47
class="unstyled text-white plausible-event-name=bannerClick"
class="unstyled flex-grow tracking-wide"
Copy link
Member

Choose a reason for hiding this comment

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

Why is plausible-event-name being removed?

Copy link
Member Author

Choose a reason for hiding this comment

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

Inadvertent removal. Fixed, thanks.

Copy link
Member

@aloisklink aloisklink left a comment

Choose a reason for hiding this comment

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

The link checker CI failures aren't related to your PR, so everything looks good to me!

@aloisklink aloisklink merged commit 5e61867 into master Jan 12, 2024
19 of 24 checks passed
@aloisklink aloisklink deleted the v3-announcement-bar branch January 12, 2024 16:20
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.

None yet

3 participants