Skip to content

build: update Angular to v17

Bc. Jakub Hejda requested to merge hejda/STR-1096/angular_17 into main

Description

build: update Angular to v17

  • Update Angular to v17 via Nx migration process.
  • Manually update also ngx-mat-select-search, eslint-plugin-unused-imports and cypress which was needed to successfully update Angular.
  • Skip control-flow migration for now, which was introduced in v17 (but migration guide is still in developer preview).
  • The new .env file was created to disable new Nx functionality Inferred Tasks. There will be need to do some changes in configuration to use this new feature and Nx is creating the migration process for existing projects, so we want to disable this new feature temporarily.

build: resolved eslint errors after Angular update to v17

build: change build system from webpack to esbuild + vite

  • The build system for all our apps changed from webpack to esbuild + vite.
  • Removed webpack-bundle-analyzer and related script from package.json file.
  • Fixed build warning in perun-utils.ts.

build: updated all other libraries after Angular update to v17

  • Updated all other libraries which were not updated by migration process.
  • Commitlint libraries were not updated cause there is still some issue with ES modules in new major version.

chore: fix errors after eslint and prettier upgrade

  • With the eslint-plugin-cypress upgrade the new rule unsafe-to-chain-command was enabled.
  • Fixed the chaining errors in our E2E tests.
  • With prettier upgrade indentation for nested ternaries was introduced.

How to test

Successful npm install, a random manual test of GUI, especially logging in/out, and testing some basic features.

Author's checklist

  • I have followed the contribution guidelines
  • This MR has been tested or does not change functionality
  • I have added relevant merge request dependencies (if this MR has any)
  • I have added the correct labels
  • I have assigned reviewers (if any are relevant)
  • I have edited the documentation (if the changes require it) or I have noted the need for the change if I do not have access to the documentation
  • I have marked all introduced BREAKING CHANGES or necessary DEPLOYMENT NOTES in the commit message(s)

Reviewer's checklist

  • This MR has been tested or does not change functionality
  • This MR has correct commit message format

Other information

All commits probably excluded build: change build system from webpack to esbuild + vite can be squashed

Related issues

Closes STR-1096

Edited by Michal Berky

Merge request reports