-
Notifications
You must be signed in to change notification settings - Fork 75
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
remove rc1 qualifier #543
remove rc1 qualifier #543
Conversation
Signed-off-by: Amit Galitzky <amgalitz@amazon.com>
Codecov Report
@@ Coverage Diff @@
## main #543 +/- ##
=========================================
Coverage 79.07% 79.07%
Complexity 4207 4207
=========================================
Files 296 296
Lines 17681 17681
Branches 1877 1877
=========================================
+ Hits 13981 13982 +1
+ Misses 2801 2800 -1
Partials 899 899
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Same question: This is only for 2.0 branch? |
Should be for main branch and then backported to 2.0. We want main to be most up to date for all incoming development so we don't want to keep main to 2.0-rc1 while we have moved on to 2.0. |
Signed-off-by: Amit Galitzky <amgalitz@amazon.com> (cherry picked from commit dbd311a)
Signed-off-by: Amit Galitzky <amgalitz@amazon.com> (cherry picked from commit dbd311a)
Signed-off-by: Amit Galitzky <amgalitz@amazon.com> (cherry picked from commit dbd311a)
Signed-off-by: Amit Galitzky amgalitz@amazon.com
Description
remove rc1 qualifier and use of
_type
in test as that field doesn't exist anymore for default index.Issues Resolved
resolves #536
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.