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

[p5.js 2.0 RFC Proposal]: use lighter dependency #7453

Open
2 of 21 tasks
asukaminato0721 opened this issue Jan 2, 2025 · 1 comment
Open
2 of 21 tasks

[p5.js 2.0 RFC Proposal]: use lighter dependency #7453

asukaminato0721 opened this issue Jan 2, 2025 · 1 comment

Comments

@asukaminato0721
Copy link
Contributor

Increasing access

p5.js use some sub-dependency which use unnecessary spaces. So use lighter dependencies.

Which types of changes would be made?

  • Breaking change (Add-on libraries or sketches will work differently even if their code stays the same.)
  • Systemic change (Many features or contributor workflows will be affected.)
  • Overdue change (Modifications will be made that have been desirable for a long time.)
  • Unsure (The community can help to determine the type of change.)

Most appropriate sub-area of p5.js?

  • Accessibility
  • Color
  • Core/Environment/Rendering
  • Data
  • DOM
  • Events
  • Image
  • IO
  • Math
  • Typography
  • Utilities
  • WebGL
  • Build process
  • Unit testing
  • Internationalization
  • Friendly errors
  • Other (specify if possible)

What's the problem?

try to reduce a bit of build size/npm install size.

What's the solution?

use https://github.com/SukkaW/nolyfill

Pros (updated based on community comments)

Example list:

  • Consistency: This proposal will not affect current behavior...
  • Size: This proposal will reduce the build/download size.

Cons (updated based on community comments)

  • require node >= 12 but currently it's already node >= 20

Proposal status

Under review

@limzykenneth
Copy link
Member

I think this is no longer necessary as we are only using eslint itself and not the referenced eslint plugins dependencies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants