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

Dominoes: Add test case #2151

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

zeltbrennt
Copy link

@zeltbrennt zeltbrennt commented Nov 16, 2022

A new test case for certain solutions that would satisfy all previous tests but would still be wrong.

Related exercism/java#2196

A new test case for certain solutions that would satisfy all previous tests but would still be wrong
@andrerfcsantos
Copy link
Member

Hi @zeltbrennt,

Thanks for opening this. Changed the wording on the PR description from "Resolves " to "Related ". With "resolves" that would cause the issue on the Java track to close when this PR is merged. We don't necessarily want that because after the PR is approved here we still need one in the Java track to update the Java tests and then you can close the issue there.

I'll let people more familiar with problem specs do the actual review on this one.

@glennj glennj changed the title Add test case Dominoes: Add test case Dec 8, 2022
@IsaacG
Copy link
Member

IsaacG commented Apr 9, 2023

Is there a forum discussion for this issue where the proposed change was discussed and given a go-ahead?

@glennj
Copy link
Contributor

glennj commented Apr 9, 2023

Is there a forum discussion for this issue where the proposed change was discussed and given a go-ahead?

I think this was created before the forum.

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.

4 participants