-
Notifications
You must be signed in to change notification settings - Fork 27
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
Linting improvements #1000
Linting improvements #1000
Conversation
Signed-off-by: Greg Schohn <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1000 +/- ##
============================================
+ Coverage 80.16% 80.30% +0.13%
- Complexity 2720 2722 +2
============================================
Files 370 365 -5
Lines 13624 13595 -29
Branches 938 939 +1
============================================
- Hits 10922 10917 -5
+ Misses 2131 2105 -26
- Partials 571 573 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
2c7b618
to
e8549eb
Compare
Signed-off-by: Greg Schohn <[email protected]>
e8549eb
to
3600c60
Compare
@@ -462,7 +451,7 @@ public String getFailureMessage() { | |||
} | |||
|
|||
public static class OperationFailed extends RfsException { | |||
public final HttpResponse response; | |||
public final transient HttpResponse response; |
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.
Interesting... i've never seen transient
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.
LGTM, thanks
Signed-off-by: Greg Schohn <[email protected]>
Signed-off-by: Greg Schohn <[email protected]> # Conflicts: # RFS/src/main/java/org/opensearch/migrations/bulkload/common/InvalidResponse.java
6ed1e5a
to
98e6d94
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.
LGTM
Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Is this a backport? If so, please add backport PR # and/or commits #
Testing
[Please provide details of testing done: unit testing, integration testing and manual testing]
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.