Skip to content

chore: upgrade to nx v22 #512

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
Open
wants to merge 55 commits into
base: main
Choose a base branch
from
Open

chore: upgrade to nx v22 #512

wants to merge 55 commits into from
+231 −1,044

Conversation

@raymondkarstensen
Copy link
Collaborator

@raymondkarstensen raymondkarstensen commented Nov 20, 2025 •
edited
Loading

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

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

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@raymondkarstensen
chore: nx migrate 20.5.0
95ab9f1
@raymondkarstensen
chore: migrating eslint config @nx/eslint:convert-to-flat-config
24c777d
@raymondkarstensen
chore: migrating eslint config @nx/eslint:convert-to-flat-config
455451d
@raymondkarstensen
chore: run migrations
14c9ada
@raymondkarstensen
chore: remove no-extra-semi lint rule as this is fixed when formattin…
2fccd38 
…g with prettier
@raymondkarstensen
fix: lint warnings and errors
c278a7a
@raymondkarstensen
chore: nx migrate latest
91a3c8a
@raymondkarstensen
chore: run migration and migrate to use of inject
7fc8dad
@raymondkarstensen
chore: run migration convert-to-standalone
f919696
@raymondkarstensen
chore: run migration prune-modules
f0386b6
@raymondkarstensen
chore: run migration standalone-bootstrap
7113d6d
@raymondkarstensen
fix: add CUSTOM_ELEMENTS_SCHEMA to allow unknown element app-mime-vie…
6228d82 
…wer for elements
@raymondkarstensen
chore: migrate to standalone
5c11483
@raymondkarstensen
fix: remove use of deprecated resolveComponentFactory when creating c…
6e51822 
…omponent
@raymondkarstensen
chore: update jest and build dependency versions
50402ce
@raymondkarstensen
fix: removed functions
cc9d943
@raymondkarstensen
chore: sort imports
05abc9f
@raymondkarstensen
fix: test
fc0f89b
@raymondkarstensen
fix: replace deprecated HttpClientTestingModule with provideHttpClien…
b334b70 
…t, and provideHttpClientTesting were needed
@raymondkarstensen
fix: shorten import warnings
bbda40a
@raymondkarstensen
fix: remove unnecessary assign to const done by migrate
76537ae
@raymondkarstensen
fix: remove modules from elements and integration
1936563
@raymondkarstensen
fix: some obvious lint errors to imports and missing readonly attributes
ed72d67
@raymondkarstensen
fix: add member-ordering to eslint and fix errors
eeb44ac
@raymondkarstensen
fix: remove modules and services from list of providers
33bc8fa 
Modules should be imported where they are needed.
Should not be necessary to provide injectable services
@raymondkarstensen
fix: not necessary to provide injectable services
8a52563
@raymondkarstensen
fix: missing providedIn in injectable annotation
b70be01
@raymondkarstensen
fix: formatting
18e1732
@raymondkarstensen
fix: eslint member-ordering errors
def6e60
@raymondkarstensen
fix: eslint errors
f68ce7f
@raymondkarstensen
fix: remove modules and services from list of providers
f7630c4 
Modules should be imported where they are needed.
Should not be necessary to provide injectable services
@raymondkarstensen
fix: cleanup unnecessary providers
35d16a2
@raymondkarstensen
fix: replace RouterTestingModule
88358d6
@raymondkarstensen
fix: add provideHttpClientTesting where needed
a5dcbe6
@raymondkarstensen
fix: delete ngx-mime module since all components now are standalone
2816028
@raymondkarstensen
fix: extract routes to dedicated files to separate them from bootstra…
cbc83e9 
…p logic
@raymondkarstensen
fix: delete ngx-mime module files
0b805b7
@raymondkarstensen
revert: changes to files in dist folder
e32ea98
@raymondkarstensen
fix: remove unnecessary browsermodule
0ce2862
@raymondkarstensen
fix: replace easeInWithDelay animation with pure css animation
2c3502b
@raymondkarstensen
fix: replace deprecated Angular animations with pure css animations
f9d9ed6
@raymondkarstensen
fix: replace deprecated Angular animations with pure css animations
eba580c
@raymondkarstensen
fix: services must be provided by component instead of in root to sup…
02256f9 
…port multiple viewers
@raymondkarstensen
fix: remove NoopAnimationsModule
df0d74d
@raymondkarstensen
fix: incorrect import path
cc3f73e
@raymondkarstensen
fix: import module for MatSidenav instead of each component
56bab06
@raymondkarstensen
Revert "fix: delete ngx-mime module files"
1b0ab71 
This reverts commit 0b805b7.
@raymondkarstensen
fix: revert removal of MimeModule
5cda25d
@raymondkarstensen
fix: update peerDependency versions for Angular
8609266
@raymondkarstensen
feat: create provider function for internationalization
efc631c
@raymondkarstensen
feat: create and export enum of locales available to be used in viewer
da8c669
@raymondkarstensen
fix: missing MimeViewerIntl as provider for elements project
0662a5c
@raymondkarstensen
chore: nx migrate to latest
622b8cc
@ronnymikalsen ronnymikalsen changed the title feat: upgrade to nx v22 chore: upgrade to nx v22 Nov 21, 2025
@raymondkarstensen
Merge branch 'main' into chore/nbweb-3193/upgrade-to-nx-22
5e04f9a 
# Conflicts:
#	.gitignore
#	apps/demo/src/app/app.config.ts
#	libs/ngx-mime/package.json
#	migrations.json
#	package.json
#	yarn.lock
@raymondkarstensen
chore: nx migrate to latest
d2098b1
@raymondkarstensen raymondkarstensen marked this pull request as ready for review November 21, 2025 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@raymondkarstensen