Skip to content
This repository was archived by the owner on Jul 22, 2025. It is now read-only.

FIX: Exclude the first solution itself when checking if an existing solution #378

Merged
merged 1 commit into from
Jul 2, 2025

Conversation

nattsw
Copy link
Contributor

@nattsw nattsw commented Jul 2, 2025

When fixing #377, we uncovered another bug where the existing check does not remember to exclude the first solution itself.

This PR

  • makes sure to exclude the first solution when checking for an existing solution
  • better tests -- the existing test does not accept the post prior to making the check.

@nattsw nattsw merged commit 041b58e into main Jul 2, 2025
6 checks passed
@nattsw nattsw deleted the exclude-itself branch July 2, 2025 03:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants