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.

Sign up for GitHub

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

Jump to bottom

[pre-commit.ci] pre-commit autoupdate #12577

Merged
merged 1 commit into from
Feb 14, 2025
Merged

[pre-commit.ci] pre-commit autoupdate #12577

merged 1 commit into from
Feb 14, 2025

Conversation

pre-commit-ci[bot]
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Feb 10, 2025

@pre-commit-ci pre-commit-ci bot requested a review from cclauss as a code owner February 10, 2025 19:53
@algorithms-keeper algorithms-keeper bot added the tests are failing Do not merge until tests pass label Feb 10, 2025
@cclauss
Copy link
Member

cclauss commented Feb 14, 2025

@anandfresh Please do not approve pull requests that have failing test.

@cclauss cclauss force-pushed the pre-commit-ci-update-config branch from c485982 to 3c7b4ea Compare February 14, 2025 07:40
@algorithms-keeper algorithms-keeper bot added merge conflicts Open a new PR or rebase on the latest commit awaiting reviews This PR is ready to be reviewed and removed tests are failing Do not merge until tests pass labels Feb 14, 2025
@cclauss cclauss force-pushed the pre-commit-ci-update-config branch from f0a2c55 to 2042b1f Compare February 14, 2025 07:41
@algorithms-keeper algorithms-keeper bot removed the merge conflicts Open a new PR or rebase on the latest commit label Feb 14, 2025
@cclauss cclauss force-pushed the pre-commit-ci-update-config branch from 1e8a126 to afdda4e Compare February 14, 2025 07:43
@algorithms-keeper algorithms-keeper bot added tests are failing Do not merge until tests pass labels Feb 14, 2025
@cclauss
Copy link
Member

cclauss commented Feb 14, 2025 •
edited
Loading

Interesting that git mv data_structures/queue data_structures/queues causes several tests to fail!

FAILED data_structures/queues/circular_queue.py::data_structures.queues.circular_queue.CircularQueue.__len__
FAILED data_structures/queues/circular_queue.py::data_structures.queues.circular_queue.CircularQueue.enqueue
FAILED data_structures/queues/priority_queue_using_list.py::data_structures.queues.priority_queue_using_list.ElementPriorityQueue
FAILED data_structures/queues/priority_queue_using_list.py::data_structures.queues.priority_queue_using_list.FixedPriorityQueue
============= 4 failed, 1950 passed, 1 warning in 76.86s (0:01:16) =============

@cclauss cclauss force-pushed the pre-commit-ci-update-config branch 2 times, most recently from 05eaa68 to 9deb95e Compare February 14, 2025 08:56
@cclauss cclauss force-pushed the pre-commit-ci-update-config branch from 9deb95e to 02b91e7 Compare February 14, 2025 09:01
@algorithms-keeper algorithms-keeper bot removed tests are failing Do not merge until tests pass labels Feb 14, 2025
@algorithms-keeper algorithms-keeper bot removed the awaiting reviews This PR is ready to be reviewed label Feb 14, 2025
@cclauss cclauss merged commit 738253e into master Feb 14, 2025
6 checks passed
@cclauss cclauss deleted the pre-commit-ci-update-config branch February 14, 2025 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cclauss cclauss cclauss approved these changes

@anandfresh anandfresh anandfresh approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cclauss @anandfresh