-
Notifications
You must be signed in to change notification settings - Fork 1
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
v2.0.0 #1442
Conversation
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.
The PR diff size of 34235 lines exceeds the maximum allowed for the inline comments feature.
This pull request introduces 1 alert when merging b857421 into 03800e1 - view on LGTM.com new alerts:
|
Update dependency eslint to v8.7.0
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.
The PR diff size of 34211 lines exceeds the maximum allowed for the inline comments feature.
Update dependency electron-log to v4.4.4
Update dependency csv-parse to v5.0.4
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.
The PR diff size of 34215 lines exceeds the maximum allowed for the inline comments feature.
This pull request introduces 1 alert when merging 2071bfb into 03800e1 - view on LGTM.com new alerts:
|
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.
The PR diff size of 8465 lines exceeds the maximum allowed for the inline comments feature.
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.
The PR diff size of 8467 lines exceeds the maximum allowed for the inline comments feature.
This pull request introduces 1 alert when merging c7c0110 into 03800e1 - view on LGTM.com new alerts:
|
@@ -108,15 +108,15 @@ describe('Create concomitant med controller test', () => { | |||
endDate: 13428904 | |||
}) | |||
.then(({ status, body }) => { | |||
expect(status).toBe(400); |
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.
This is a breaking change and will have to enforce a major version number release
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.
The PR diff size of 8485 lines exceeds the maximum allowed for the inline comments feature.
This pull request introduces 1 alert when merging ed565cd into 03800e1 - view on LGTM.com new alerts:
|
fix: express rate limit is not a constrctor
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.
The PR diff size of 8487 lines exceeds the maximum allowed for the inline comments feature.
Code Climate has analyzed commit b7acfcc and detected 25 issues on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
This pull request introduces 1 alert when merging b7acfcc into 03800e1 - view on LGTM.com new alerts:
|
This is a PR for the release of
v2.0.0