Skip to content

Conversation

@forman
Copy link
Member

@forman forman commented Jan 17, 2025

Added two time-related rules.

Closes #15.

Checklist (strike out non-applicable):

  • Changes documented in CHANGES.md
  • Related issue exists and is referred to in the PR description and CHANGES.md
  • Added docstrings and API docs for any new/modified user-facing classes and functions
  • Changes/features documented in docs/*
  • Unit-tests adapted/added for changes/features
  • Test coverage remains or increases (target 100%)

@codecov
Copy link

codecov bot commented Jan 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.24%. Comparing base (1021929) to head (33f7bf8).
Report is 18 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #24      +/-   ##
==========================================
+ Coverage   97.16%   97.24%   +0.07%     
==========================================
  Files          53       55       +2     
  Lines        2117     2174      +57     
  Branches      286      298      +12     
==========================================
+ Hits         2057     2114      +57     
  Misses         32       32              
  Partials       28       28              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@forman forman marked this pull request as ready for review January 18, 2025 08:28
@forman forman requested a review from pont-us January 18, 2025 08:31
@forman forman self-assigned this Jan 18, 2025
@forman
Copy link
Member Author

forman commented Jan 20, 2025

@pont-us , please have a look when there is more time.

@forman forman merged commit b0b61db into main Jan 20, 2025
6 checks passed
@forman forman deleted the forman-15-time_rules branch January 20, 2025 13:37
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.

Add a rule to enforce restrictions on time co-ordinates

2 participants