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

Add a test for issue #676 #695

Merged
merged 1 commit into from
Sep 2, 2024

Conversation

wilfwilson
Copy link
Collaborator

I've added a test for a version of the bug that @fingolfin reported in #676; see #678 which gave the fix.

@james-d-mitchell james-d-mitchell changed the base branch from stable-1.8 to main September 2, 2024 07:43
@james-d-mitchell
Copy link
Member

Rebased onto main (the next release will be 1.9.0) to fix the gaplint job. Thanks @wilfwilson !

@james-d-mitchell james-d-mitchell added the tests A label for issues/PRs relating to adding or updating tests or the testing system label Sep 2, 2024
This concerns using DigraphAllSimpleCircuits with
digraphs with non-standard vertex labels.
Copy link
Member

@james-d-mitchell james-d-mitchell left a comment

Choose a reason for hiding this comment

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

Looks good!

@james-d-mitchell james-d-mitchell merged commit 1a59c6e into digraphs:main Sep 2, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests A label for issues/PRs relating to adding or updating tests or the testing system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants