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

Implement more convenient delegate methods for Windows and Interval #1644

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

JoelCourtney
Copy link
Contributor

@JoelCourtney JoelCourtney commented Mar 13, 2025

  • Tickets addressed: AERIE-000
  • Review: By commit
  • Merge strategy: Merge (no squash)

Description

A few quick updates to Windows and Interval to address comments from @ewferg. Adds some overloads to Windows and Interval to make interop between them easier. Also adds starts, ends and shiftEndpoints to `Windows, which I apparently forgot to implement.

Verification

  • I'll add some tests for starts, ends, and shiftEndpoints. The rest are trivial delegations and don't need new tests.

Documentation

All these functions have doc comments

Future work

Sorry, something went wrong.

@JoelCourtney JoelCourtney requested a review from ewferg March 13, 2025 18:48
@JoelCourtney JoelCourtney self-assigned this Mar 13, 2025
@JoelCourtney JoelCourtney requested a review from a team as a code owner March 13, 2025 18:48
@JoelCourtney JoelCourtney removed the request for review from adrienmaillard March 13, 2025 18:48
@JoelCourtney
Copy link
Contributor Author

JoelCourtney commented Mar 13, 2025

@sonarqubecloud

Quality Gate failed

Failed conditions
21.5% Being a total nerd (required ≤ 3%)

See analysis details on Youtube

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
20.9% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

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.

None yet

1 participant