-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
Vue 2.7 refresh #2
Conversation
Hmm, so neflify checks failed as @vue/cli-service@5.0.8 requires node >= 12, and those checks are running on node 10. Oops. |
Thanks for the PR! I added a |
This reverts commit 6069136.
Oops, I was a bit fast with this one. The icons are much too big now. Care to investigate? |
I filed an updated PR, which corrects the icon size problem (pending a real fix in fa). |
Hi,
First, thanks for writing this app. It's pretty nice!
Anyways, I tried building this with node18, and it fails to compile. After dropping back to node14, it would compile, but only if I didn't change package-lock.json. So, I eventually did a refresh, fixed some eslint bugs, and everything is update, using vue 2.7.14 and @fortawesome/vue-fontawesome@2.
The icons are a little larger with this version though. I'm not so familiar with FA, but maybe I'll look more at this change.
You also might not want the automatic lint changes. I so, I can drop that last commit from this PR.
Cheers,
John