Skip to content

Conversation

YarShev
Copy link
Collaborator

@YarShev YarShev commented Sep 6, 2024

What do these changes do?

  • first commit message and PR title follow format outlined here

    NOTE: If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

  • passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
  • passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
  • signed commit with git commit -s
  • Resolves Error: No artifacts found matching pattern 'coverage-data-*' #7389
  • tests passing
  • module layout described at docs/development/architecture.rst is up-to-date

anmyachev
anmyachev previously approved these changes Sep 6, 2024
@YarShev YarShev force-pushed the dev/yigoshev-issue7389 branch from f42ce2f to 65a4a5a Compare September 6, 2024 23:38
Signed-off-by: Igoshev, Iaroslav <[email protected]>
Signed-off-by: Igoshev, Iaroslav <[email protected]>
@YarShev YarShev force-pushed the dev/yigoshev-issue7389 branch from 5b67b35 to ceda52f Compare September 9, 2024 09:16
@YarShev YarShev force-pushed the dev/yigoshev-issue7389 branch from 81cd23b to 6ac0aec Compare September 9, 2024 11:44
@YarShev YarShev merged commit 3357709 into modin-project:main Sep 9, 2024
36 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.

Error: No artifacts found matching pattern 'coverage-data-*'

2 participants