Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

fix(indicator): Exhaustively check for valid stateChanges subject bef… #2800

Merged
merged 1 commit into from
Dec 22, 2023

Conversation

tomheller
Copy link
Collaborator

…ore completing it.

Pull Request


Hi, thank you for contributing to Barista with this pull request (PR).

To ensure a fast process and merging of your PR please make sure it fulfills the
coding standards and contribution guidelines.

  • A feature proposal has been provided, discussed and approved first.
  • There is a meaningful description of the issue in GitHub (Screenshots are
    often helpful).
  • If the PR introduces breaking-changes or deprecations it matches the following
    guidelines.
    • The commit message follows our commit guidelines.
    • Tests for the changes have been added (for bug fixes / features).
    • Docs have been added / updated (for bug fixes / features).

Please choose the type appropriate for the changes below:

Type of PR

Checklist

  • I have read the CONTRIBUTING doc and I follow the PR guidelines
  • Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

@tomheller tomheller requested a review from ffriedl89 as a code owner December 19, 2023 10:06
Copy link

Deploy preview for barista ready!

✅ Preview
https://barista-idst8003y-dynatrace-oss.vercel.app

Built with commit b7eb876.
This pull request is being automatically deployed with vercel-action

@tomheller tomheller merged commit f8abf13 into master Dec 22, 2023
11 of 12 checks passed
@tomheller tomheller deleted the additional-complete-throw-checks branch December 22, 2023 11:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant