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

[WIP] Numbits #187

Closed
wants to merge 3 commits into from
Closed

[WIP] Numbits #187

wants to merge 3 commits into from

Conversation

baldawar
Copy link
Collaborator

Issue #, if available: #179

Description of changes:

WIP change to test the new benchmakrs get auto-triggerred.
Change is pending proper description, readme, and version bump.

Benchmark / Performance (for source code changes):

<replace this with output from /src/test/software/amazon/event/ruler/Bechmarks.java here.

The benchmark results can be fetched from "Pull request checks -> Java build -> build (ubuntu-X.Y, 8) -> Run benchmarks".>

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

For each test within the class we add a list of "noMatches" objects
that should never be pattern matches intentinoally or not. However
we never check if we ever accidentally register these pattern when
adding other patterns. We will fix this by tracking when these
patterns get added (probably by error) and then later validate the
checks

Note, no impact to the correctness of the ByteMachine itself, just
that the test scaffolding was broken and needed to be fixed.

See #181 for more context.
@baldawar baldawar closed this Sep 18, 2024
@baldawar baldawar deleted the numbits branch September 26, 2024 22:55
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