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

Simplify Dependency management #402

Open
dmohns opened this issue Sep 17, 2024 · 1 comment
Open

Simplify Dependency management #402

dmohns opened this issue Sep 17, 2024 · 1 comment

Comments

@dmohns
Copy link
Contributor

dmohns commented Sep 17, 2024

Reason: The project uses Rush for the backend part. The frontend part was moved into another repository, but the code was kept in the backend repo (see above).

This makes usage of Rush obsolete (Rush was designed to manage mono-repos of many interdependent projects) and complicates the setup a lot.

Rush seems to be configured to use PNPM. There are some build instructions using yarn and npm is used as well.

Also, the Node version isn’t pinned to the minor version. The Dockerfiles just say node:20-alpine So, upgrades might happen unintentionally.

Steps:

  • Consolidate all build steps, local build etc.. to use a single Node package manager.
  • Remove Rush and all rush related configuration files.

Definition of done: The frontend and backend application are managed using a single package manager. As a developer, the only prerequisite is to have the correct dependency manager installed.

@dmohns
Copy link
Contributor Author

dmohns commented Sep 30, 2024

This has a dependency on the build pipeline. We can only start working on it, once we have full control over how the project is built and deployed.

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

1 participant