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

Print empty lists with two spaces for consistency #994

Merged
merged 1 commit into from
Feb 16, 2024

Conversation

zickgraf
Copy link
Contributor

@zickgraf zickgraf commented Feb 5, 2024

as a preparation for gap-system/gap#5418

@zickgraf zickgraf force-pushed the main branch 2 times, most recently from 7c0d445 to 500d536 Compare February 5, 2024 15:13
@james-d-mitchell
Copy link
Collaborator

@zickgraf I'll be happy enough to merge the PR's you've raised in Digraphs and Semigroups, but I'm reluctant to do that before this PR is merged. I'll mention this in those PR's also

@james-d-mitchell james-d-mitchell added gap-compatibility A label for PRs or issues that are related to compatibility with changes in GAP do not merge Label for PR that should not be merged labels Feb 6, 2024
@zickgraf
Copy link
Contributor Author

As expected, the tests are now failing with GAP master because gap-system/gap#5418 was merged, and this PR should hopefully fix the issues :-)
(I have rebased to retrigger the CI but the checks require approval.)

@james-d-mitchell james-d-mitchell enabled auto-merge (rebase) February 16, 2024 18:43
@james-d-mitchell james-d-mitchell merged commit 3a1e71c into semigroups:main Feb 16, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Label for PR that should not be merged gap-compatibility A label for PRs or issues that are related to compatibility with changes in GAP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants