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

[Breaking Change] Fix #526: Detect package.json changes for install #527

Merged
merged 3 commits into from
Sep 29, 2023

Conversation

melloware
Copy link
Contributor

@melloware melloware commented Sep 28, 2023

Fix #526: Detect package.json changes for install
Fix #503: Logging freeze in dev mode (quarkusio/quarkus#36086 (comment))

The breaking change is that the quinoa target build directory is not target/quinoa/build instead of target/quinoa-build. This may affect automation using the target quinoa directory.

On a fresh build the package.json doesn't exist in /target/quinoa so it triggers an install.

Subsequent builds compare the two files for equality if anything has changed it triggers an install.

@melloware
Copy link
Contributor Author

@ia3andy need some help with the 1 remaining failing unit test I don't quite understand how the test works.

Copy link
Collaborator

@ia3andy ia3andy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is awesome!

I fixed a few stuff and added a few other stuff:

  • Checking size before reading the content (a bit more efficient in most cases)
  • Reading the bytes is enough (no need to convert to String)
  • Remove old dev-ui install (not necessary anymore)
  • Remove lockfile dev-reload because it was causing loops (this was for dev-ui install)
  • Added package.json change detection to proxy mode

For some reason this fixed the freezing issue 🎉 #503 (I found out why see issue)

@ia3andy ia3andy changed the title Fix #526: Detect package.json changes for install [Breaking Change] Fix #526: Detect package.json changes for install Sep 29, 2023
@ia3andy ia3andy merged commit 739702d into quarkiverse:main Sep 29, 2023
2 checks passed
@melloware melloware deleted the Q526 branch September 29, 2023 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants