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

build(nx): setup workspace & migrate packages #476

Closed
wants to merge 147 commits into from

Conversation

marcus-sa
Copy link
Contributor

@marcus-sa marcus-sa commented Sep 10, 2023

Summary of changes

The following packages have been migrated to Nx

  • angular-universal
  • api-console-api
  • api-console-gui
  • api-console-module
  • app
  • core
  • broker
  • bson
  • core
  • core-rxjs
  • create-app
  • crypto
  • desktop-ui
  • event
  • framework
  • framework-debug-api
  • framework-debug-gui
  • http
  • injector
  • logger
  • mongo
  • mysql
  • orm
  • orm-browser
  • orm-browser-api
  • orm-browser-gui
  • orm-browser-module
  • orm-integration
  • postgres
  • rpc
  • rpc-tcp
  • sql
  • sqlite
  • stopwatch
  • template - some tests are failing
  • topsort
  • type - generic/infer tests are failing
  • type-angular
  • type-compiler
  • type-spec
  • ui-library
  • vite
  • workflow

https://nx.dev/packages/rollup/executors/rollup is currently being used for building packages.

We can add automated changelogs, versioning & package publishing using https://github.com/changesets/changesets or https://github.com/release-it/release-it

Relinquishment of Rights

Please mark following checkbox to confirm that you relinquish all rights of your changes:

  • I waive and relinquish all rights regarding this changes (including code, text, and images) to Deepkit UG (limited), Germany. This changes (including code, text, and images) are under MIT license without name attribution, copyright notice, and permission notice requirement.

peer-deps-external conflicts with nx-resolve-packages when @deepkit packages are specified in package.json peerDependencies
marcus-sa and others added 23 commits September 13, 2023 16:16
Signed-off-by: Marcus S. Abildskov <8391194+marcus-sa@users.noreply.github.com>
@marcus-sa marcus-sa marked this pull request as ready for review October 2, 2023 13:22
Signed-off-by: Marcus S. Abildskov <8391194+marcus-sa@users.noreply.github.com>
@marcus-sa marcus-sa closed this Oct 22, 2023
@marcus-sa
Copy link
Contributor Author

marcus-sa commented Oct 22, 2023

It was decided not to use Nx because it's too complex and requires a lot of configuration

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.

1 participant