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

.github: update s3 tests run step #837

Merged
merged 1 commit into from
Oct 6, 2023

Conversation

evgeniiz321
Copy link

In order to support nspcc-dev/s3-tests#43. There is just one set of tests, it runs for 8min30sec, there is no point of having two different sets of tests right now.

@codecov
Copy link

codecov bot commented Oct 6, 2023

Codecov Report

Merging #837 (5629e37) into master (977a16c) will not change coverage.
Report is 1 commits behind head on master.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #837   +/-   ##
=======================================
  Coverage   33.90%   33.90%           
=======================================
  Files          62       62           
  Lines       10519    10519           
=======================================
  Hits         3566     3566           
  Misses       6552     6552           
  Partials      401      401           

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

@vvarg229
Copy link
Collaborator

vvarg229 commented Oct 6, 2023

image
How is it that the tests were cut by 1 hour?

@evgeniiz321
Copy link
Author

Failing tests were the reason of 1 hour run. Without them - it is much faster.

Signed-off-by: Evgeniy Zayats <[email protected]>
@roman-khimov roman-khimov merged commit a283902 into nspcc-dev:master Oct 6, 2023
12 of 14 checks passed
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.

3 participants