Skip to content

chore(general): update NX, dependencies, and eslint config #537

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

Merged
merged 1 commit into from
Jul 14, 2025

Conversation

EricPoul
Copy link
Contributor

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[x] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[x] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[x] Other... Please describe: Dependencies update

What is the current behavior?

Issue Number: #522

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@EricPoul
Copy link
Contributor Author

@NetanelBasal I didn't update versions. Don't know how you're used to doing it.
HTML is auto-updated with the latest NX and ESLint rules. I've tried to update everything possible that could throw a deprecation warning while installing elf store.

@NetanelBasal NetanelBasal merged commit da79533 into ngneat:master Jul 14, 2025
1 check passed
@NetanelBasal
Copy link
Member

Thanks!

@EricPoul
Copy link
Contributor Author

@NetanelBasal, probably you should redeploy so I can install in my project and check what else needs updating

@NetanelBasal
Copy link
Member

Do we need to publish all packages?

@EricPoul
Copy link
Contributor Author

It looks like cli is enough for now since it used old @oclif and after I will be able to check what else should be updated and p[ublished

@NetanelBasal
Copy link
Member

Done

@EricPoul
Copy link
Contributor Author

EricPoul commented Jul 16, 2025

Looks fine, You can close the issue. If something else comes up, I will update it later.

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.

2 participants