missing test coverage (possibly): `pop!`/`popfirst!` on `Deque`/`CircularDeque`
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 55/100
Research direction
Start by reading issue #986 and PR #991, then inspect the DataStructures tests for Deque and CircularDeque and run the nightly Julia CI job. Done means coverage exercises pop! and popfirst! for both types and catches the reported regression.
Written by the indexing model from the issue text.
Description
Issue #986 is not caught by the DataStructures CI, even when a job is run for nightly Julia. See PR #991, on which all checks pass, even though PR #991 adds nightly Julia CI configurations. That implies the DataStructures test suite may need better coverage.
- Dominant language
- Julia
- Stars
- 747
- Forks
- 261
- Avg merge
- 12d 6h
- Merged PRs (30d)
- 1
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from JuliaCollections/DataStructures.jl
-
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
-
append!(l1::MutableLinkedList, l2::MutableLinkedList) fails for empty l2Possibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
-
Difficulty 4/5 3-5 days Newbie friendliness 30/100
JuliaCollections/DataStructures.jl#997 · 1 comment ·
-
several `mutable struct` type definitions do not require mutability, review and adjustPossibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 4/5 3-5 days Newbie friendliness 45/100
-
a slighly uncanny `UndefRefError` with default-empty SortedSetPossibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 3/5 1-2 days Newbie friendliness 52/100
JuliaCollections/DataStructures.jl#969 · 6 comments ·
All issues in JuliaCollections/DataStructures.jl
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
JuliaGPU/KernelAbstractions.jl#897 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
oscar-system/GAP.jl#1455 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
JuliaAI/MLJTransforms.jl#85 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
SciML/OrdinaryDiffEq.jl#4875 ·
Maintainers usually reply within 4 days