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

nx monorepo cleanup #117

Open
14 of 29 tasks
43081j opened this issue Sep 16, 2024 · 10 comments
Open
14 of 29 tasks

nx monorepo cleanup #117

43081j opened this issue Sep 16, 2024 · 10 comments

Comments

@43081j
Copy link
Collaborator

43081j commented Sep 16, 2024

eslint-plugin

esbuild

js

next

nx

webpack

plugin

react

react-native

rollup

vite

vue

web

create-nx-plugin

expo

monorepo


if anyone tackles any of these, let me know and I'll update the post!

@ziebam
Copy link

ziebam commented Sep 25, 2024

An ongoing PR replacing fs-extra with native functionality in the core package: nrwl/nx#28019.

@43081j
Copy link
Collaborator Author

43081j commented Sep 25, 2024

awesome I've added it to the original post too 👍

@ziebam
Copy link

ziebam commented Sep 25, 2024

Another one in the esbuild package: nrwl/nx#28113, the expo package: nrwl/nx#28117, the react-native package: nrwl/nx#28131, and the plugin package: nrwl/nx#28133.

@43081j
Copy link
Collaborator Author

43081j commented Sep 27, 2024

great work 🎉

have updated the list

@Namchee
Copy link

Namchee commented Oct 5, 2024

We can drop strip-ansi too by the way.

@43081j
Copy link
Collaborator Author

43081j commented Oct 6, 2024

You're right!

If you or anyone else gets the time to find usages of it, please do list them here and I'll add to the original post 🙏

@Namchee
Copy link

Namchee commented Oct 6, 2024

After reading the code a bit, I think there's only one instance of strip-ansi usage that's not a fixture:

  • scripts/documentation/utils.ts

I'll try to replace this.

@JamesHenry
Copy link

Thanks so much again, folks! I love this effort.

I merged nrwl/nx#28315 and nrwl/nx#28778

@JamesHenry
Copy link

I looked into the spinner change and reported a blocking bug here: usmanyunusov/nanospinner#32

@benmccann
Copy link

benmccann commented Dec 1, 2024

js
migrate fast-glob to fdir or tinyglobby (package.json)

Here's a PR for that one: nrwl/nx#29141

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

No branches or pull requests

5 participants