-
Notifications
You must be signed in to change notification settings - Fork 69
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
Closes #1507 - Update minor versions, patches and some major versions of Config Server UI dependencies #1510
Closes #1507 - Update minor versions, patches and some major versions of Config Server UI dependencies #1510
Conversation
…into dependencies-update
versions(inspectIT#1507) Highlight enums with String values correctly (inspectIT#1488) * Treat enums with values-field differently * Fix default config exporters.yml * Fix order of possible values in test * Use toString instead of getName * Fix JSON parsing of Maps in test Using toString, empty values could not be parsed correctly by gson update patches update minor versions update eslint dependencies update @babel/core dependency re-update eslint update primeinons update primeflex update react-timeago update react-syntax-highligter update react-redux
* Activate formatter tags by default * Add indentation settings for JS * Add to CONTRIBUTING regarding JavaScript linting * Fix eol linting of .js files on Windows * Add yarn lint and format:write to README * Add section in CONTRIBUTING.md * Add to CONTRIBUTING based on comments * Fix linting command in README Co-authored-by: Heiko Holz <heiko.holz@novatec-gmbh.de>
Only one lock-file should be in project, which is yarn.lock in our case.
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.
Reviewed 19 of 22 files at r7, 4 of 4 files at r8, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @salim-1997)
10a48c0
to
45e12fc
Compare
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.
Reviewed 8 of 8 files at r9, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @salim-1997)
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.
Reviewed 2 of 2 files at r10, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @salim-1997)
Closes #1507
This change is