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.

Sign up for GitHub

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

Jump to bottom

MomentJS dependency #781

Closed
ngoov opened this issue Jan 16, 2019 · 2 comments
Closed

MomentJS dependency #781

ngoov opened this issue Jan 16, 2019 · 2 comments
Labels
Type: New Shape Request for new shape

Comments

@ngoov
Copy link

ngoov commented Jan 16, 2019

Can't the momentjs dependency be removed? It seems to be used for simple 3 date functions which are supported by the default Date() object in javascript, while using 500kb of space.
@stale
Copy link

stale bot commented Jun 29, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the Type: New Shape Request for new shape label Jun 29, 2019
@knsv
Copy link
Collaborator

knsv commented Jun 30, 2019

This has been fixed. mermaid uses moment-mini instead.

@knsv knsv closed this as completed Jun 30, 2019
@dependabot dependabot bot mentioned this issue Mar 16, 2021
Closed
mgenereu pushed a commit to mgenereu/mermaid that referenced this issue Jun 25, 2022
@sidharthv96
Merge pull request mermaid-js#781 from mermaid-js/dependabot/docker/d…
65dd5a2 
…evelop/node-18.1.0

chore(deps): bump node from 18.0.0 to 18.1.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Type: New Shape Request for new shape
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@knsv @ngoov