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

Upgraded NPM packages #212

Merged
merged 2 commits into from
Jul 22, 2024
Merged

Upgraded NPM packages #212

merged 2 commits into from
Jul 22, 2024

Conversation

crazyscientist
Copy link
Collaborator

  • Replaced vue-cli with vite
  • Adjusted import statements and Dockerfiles
  • Moved index.html
  • Declared all build targets in Makefile as phony so that -B option is not required
  • Updated README

* Replaced `vue-cli` with `vite`
* Adjusted import statements and Dockerfiles
* Moved index.html
* Declared all build targets in Makefile as phony
  so that `-B` option is not required
* Updated README
Copy link
Contributor

@s-leon s-leon left a comment

Choose a reason for hiding this comment

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

I see you also took my very nitpicky suggestion 😆 - LGTM 👍

@s-leon s-leon merged commit 098a1d1 into SUSE:master Jul 22, 2024
2 checks passed
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.

2 participants