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

Release/10.3.0 #4669

Merged
merged 364 commits into from
Jul 26, 2023
Merged

Release/10.3.0 #4669

merged 364 commits into from
Jul 26, 2023

Conversation

sidharthv96
Copy link
Member

@sidharthv96 sidharthv96 commented Jul 26, 2023

sidharthv96 and others added 30 commits July 1, 2023 00:39
Added cypress container (from their official image),
may be reconsidered later to choose our own
The only minor problem is node JS version mistmatch
package.json 18.16.0
docker-compose mermaid 18.16.1
docker-compose cypress 18.16.0

Host option in cypress docker container must be removed
in favor of possible configuration option.
http://localhost:9000 are currently hard-coded, that is bad

Updated ./run script with better documentation
and added some styles too it as well

Started sankey.spec.js as an example
Fix relative link to theme variables list
…on_destruction

Feature/1838 actor creation destruction
Replace the TypeScript `enum {a = "a", b = "b"}` types with
TypeScript's literal types (e.g. `"a" | "b"`).

This is because TypeScript enums are
[_not_ a type-level addition to JavaScript][1], and even the official
TypeScript docs say to be careful when using.

[1]: https://www.typescriptlang.org/docs/handbook/2/everyday-types.html#enums
[`const` assertions where added in TypeScript 3.4][1] and can be used
to create enum-like objects in plain JavaScript, that act like
TypeScript enums, but has none of the downsides of TypeScript enums.

[1]: https://devblogs.microsoft.com/typescript/announcing-typescript-3-4/#const-assertions
[TypeScript enums][1] are an unusual TypeScript feature, because it's
one of the only features that is "not a type-level extension of
JavaScript".

This means TypeScript generates custom code for enums, which can cause a
bunch of issues, especially as this custom code can be built differently
depending on which bundler you use, and because of this, many people
discourage the use of enums:

- [The Dangers of TypeScript Enums][2]
- [Tidy TypeScript: Prefer union types over enums][3]
- [TypeScript: Handbook - Enums # Objects vs Enums][4]

I've added an ESLint rule that forbids TypeScript enums, as in most
cases, it's better to use string literals instead, e.g. like
`type a = "a" | "b" | "c";`.

[1]: https://www.typescriptlang.org/docs/handbook/enums.html#string-enums
[2]: https://dev.to/azure/the-dangers-of-typescript-enums-55pd
[3]: https://fettblog.eu/tidy-typescript-avoid-enums/
[4]: https://www.typescriptlang.org/docs/handbook/enums.html#objects-vs-enums
Remove all TypeScript enums and forbid them in ESLint
sidharthv96 and others added 22 commits July 24, 2023 07:01
chore(deps): update all patch dependencies (patch)
…ons-list

added Typora to integrations list
Already checked in lint.
…maid into sidv/splitUnicode

* 'sidv/splitUnicode' of https://github.com/mermaid-js/mermaid:
  Update packages/mermaid/src/rendering-util/types.d.ts
Split formatted markdown strings with unicode support.
…js/mermaid into develop

* 'develop' of https://github.com/mermaid-js/mermaid: (21 commits)
  Add comments
  Update packages/mermaid/src/rendering-util/types.d.ts
  Remove redundant test. Already checked in lint.
  Update docs
  Fix lint
  Update packages/mermaid/src/rendering-util/splitText.spec.ts
  Add comments
  Add tests without Intl.Segmenter
  Cleanup
  Fix flowchart failure
  Handle proper formatting for markdown strings
  createText to TS
  Use joiner to split unicode
  Add types
  rename handle-markdown-text
  Add logs
  Cleanup
  Use splitLineToFitWidth function
  Update docs
  feat: split unicode properly
  ...

* 'develop' of https://github.com/mermaid-js/mermaid: (21 commits)
  Add comments
  Update packages/mermaid/src/rendering-util/types.d.ts
  Remove redundant test. Already checked in lint.
  Update docs
  Fix lint
  Update packages/mermaid/src/rendering-util/splitText.spec.ts
  Add comments
  Add tests without Intl.Segmenter
  Cleanup
  Fix flowchart failure
  Handle proper formatting for markdown strings
  createText to TS
  Use joiner to split unicode
  Add types
  rename handle-markdown-text
  Add logs
  Cleanup
  Use splitLineToFitWidth function
  Update docs
  feat: split unicode properly
  ...
* develop:
  Add MERMAID_RELEASE_VERSION in sankey
@sidharthv96 sidharthv96 changed the base branch from develop to master July 26, 2023 05:52
* master:
  chore: Rename C4C to C4
  update latest news section
  Corrects name of C4 link
  Bump zenuml package version
@codecov
Copy link

codecov bot commented Jul 26, 2023

Codecov Report

Merging #4669 (9c92f77) into master (db9051b) will decrease coverage by 30.71%.
The diff coverage is 70.33%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           master    #4669       +/-   ##
===========================================
- Coverage   76.03%   45.32%   -30.71%     
===========================================
  Files         137       52       -85     
  Lines       16296     6645     -9651     
  Branches      496       18      -478     
===========================================
- Hits        12390     3012     -9378     
+ Misses       3788     3633      -155     
+ Partials      118        0      -118     
Flag Coverage Δ
e2e ?
unit 45.32% <70.33%> (-12.68%) ⬇️

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

Files Changed Coverage Δ
packages/mermaid/src/mermaidAPI.ts 43.75% <26.66%> (-17.95%) ⬇️
...s/mermaid/src/diagram-api/diagram-orchestration.ts 30.23% <33.33%> (-9.06%) ⬇️
packages/mermaid/src/accessibility.ts 61.53% <40.90%> (-15.61%) ⬇️
packages/mermaid/src/diagrams/gantt/ganttDb.js 76.60% <66.66%> (-7.18%) ⬇️
...ages/mermaid/src/diagrams/sankey/sankeyDetector.ts 75.00% <75.00%> (ø)
packages/mermaid/src/config.ts 58.73% <100.00%> (-13.67%) ⬇️
packages/mermaid/src/defaultConfig.ts 43.82% <100.00%> (-49.99%) ⬇️

... and 126 files with indirect coverage changes

@knsv knsv merged commit 165e0b6 into master Jul 26, 2023
22 of 23 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.

None yet