Skip to content
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

innerring: do not panic in Head #2069

Merged
merged 1 commit into from
Nov 17, 2022

Conversation

fyrchik
Copy link
Contributor

@fyrchik fyrchik commented Nov 17, 2022

Signed-off-by: Evgenii Stratonikov evgeniy@morphbits.ru

@sami-nspcc
Copy link

I have recieved a notification of a new pull request. I am starting the build of images and binaries for further testing.

@sami-nspcc
Copy link

I am running integration tests

@codecov
Copy link

codecov bot commented Nov 17, 2022

Codecov Report

Merging #2069 (75b371d) into support/v0.34 (402bbba) will not change coverage.
The diff coverage is n/a.

❗ Current head 75b371d differs from pull request most recent head 84acee4. Consider uploading reports for the commit 84acee4 to get more accurate results

@@              Coverage Diff               @@
##           support/v0.34    #2069   +/-   ##
==============================================
  Coverage          30.67%   30.67%           
==============================================
  Files                380      380           
  Lines              28071    28071           
==============================================
  Hits                8610     8610           
  Misses             18722    18722           
  Partials             739      739           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

carpawell
carpawell previously approved these changes Nov 17, 2022
Copy link
Member

@carpawell carpawell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deserves a changelog record.

Also, the commit msg could be improved.

acid-ant
acid-ant previously approved these changes Nov 17, 2022
Signed-off-by: Evgenii Stratonikov <evgeniy@morphbits.ru>
@sami-nspcc
Copy link

I have recieved a notification of a new pull request. I am starting the build of images and binaries for further testing.

@sami-nspcc
Copy link

I am running integration tests

@sami-nspcc
Copy link

Test run is finished. Please download the tarball from st*.t5.fs.neo.org: FszwwAx3VznDysQGxBUMRj1zj3zp8VcohorVDnunrviV/BvjHzzFJnUbh4PMFkjHBVQeRE4infUz4b2EevuTjgDA7 Untar and use allure open to watch the report

@sami-nspcc
Copy link

Test run is finished. Please download the tarball from st*.t5.fs.neo.org: FszwwAx3VznDysQGxBUMRj1zj3zp8VcohorVDnunrviV/ddNXDovcMShnTmRcBiScFmEnPexni22d5pARvR19Fek Untar and use allure open to watch the report

@fyrchik fyrchik merged commit 15d2091 into nspcc-dev:support/v0.34 Nov 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants