-
Notifications
You must be signed in to change notification settings - Fork 68
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
Upgrade Dependency for Issue Found #375
Conversation
positiveVibes4all
commented
Sep 16, 2022
- Library upgrade to fix dark tooltips that are very hard to read.
- Due to some more tweaks needed on mobile this is being issued as a separate PR.
- Library upgrade to fix dark tooltips that are very hard to read. - Due to some more tweaks needed on mobile this is being issued as a separate PR.
- Conflict in auto-update of libraries found in regard to webpack and karma-webpack libraries. - Ignoring yarn lock files as NPM is currently used, and versioning package-lock files for NPM to solve this problem.
- Pushed version and added change log update.
@@ -40,7 +40,7 @@ jspm_packages | |||
lib | |||
|
|||
# Package lock files | |||
package-lock.json | |||
yarn.lock |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should track these
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Currently this repo uses NPM and this PR actually started tracking package-lock due to some auto upgrade issues that were happening. When we start using yarn, the exception for it should be removed to properly track the file though to your point.