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

Latest 2.13.x with Stepper and Accumulator tests. #28

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Ichoran
Copy link
Contributor

@Ichoran Ichoran commented Apr 16, 2019

Extra tests including Stepper and Accumulator.

@SethTisue
Copy link
Member

@Ichoran are you planning to return to this?

@Ichoran
Copy link
Contributor Author

Ichoran commented Nov 16, 2020

Oh. Um. I didn't realize the tests were failing. I'll try to get to it over the weekend.

@SethTisue
Copy link
Member

(merged in a 2.13.1 -> 2.13.3 upgrade)

val LISTBUF_PIP_11438 = F // ListBuffer throws an exception on a no-op patchInPlace
val QUEUE_SLIDE_11440 = F // Queue and ArrayStack will not give you an underfull sliding window (everything else does)
val PQ_MIP_NPE_11439 = F // Priority Queue can just give null when empty!
val LISTBUF_PIP_11438 = X // ListBuffer throws an exception on a no-op patchInPlace
Copy link
Contributor

@NthPortal NthPortal Dec 28, 2020

Choose a reason for hiding this comment

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

these all appear to be fixed

@Ichoran
Copy link
Contributor Author

Ichoran commented Dec 30, 2020

Oops, I got sick instead of getting to this, and then forgot about it. I'll try to get to it in the next couple of weeks.

Base automatically changed from master to main March 22, 2021 19:44
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