Skip to content

Fix consecutive constraint#354

Closed
FBumann wants to merge 1 commit into
feature/v3/mainfrom
feature/v3/fix-consecutive-constraint
Closed

Fix consecutive constraint#354
FBumann wants to merge 1 commit into
feature/v3/mainfrom
feature/v3/fix-consecutive-constraint

Conversation

@FBumann

@FBumann FBumann commented Sep 24, 2025

Copy link
Copy Markdown
Member

Description

Fix the formulation of a constraint. It's not really bug, as it doesn't inflict any problems.
Test need to be updated.

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Code refactoring

Related Issues

Closes #(issue number)

Testing

  • I have tested my changes
  • Existing tests still pass

Checklist

  • My code follows the project style
  • I have updated documentation if needed
  • I have added tests for new functionality (if applicable)

@coderabbitai

coderabbitai Bot commented Sep 24, 2025

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/v3/fix-consecutive-constraint

Comment @coderabbitai help to get the list of available commands and usage tips.

@FBumann
FBumann force-pushed the feature/v3/fix-consecutive-constraint branch from 631d1af to 6aa1539 Compare September 24, 2025 13:21
@FBumann

FBumann commented Jul 21, 2026

Copy link
Copy Markdown
Member Author

The consecutive-duration constraints have since been rewritten (most recently in #729), so this 1-line reformulation no longer applies as-is.

@FBumann FBumann closed this Jul 21, 2026
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.

1 participant