Skip to content

Add passing-status-required unit tests#9

Merged
dommmin-ic merged 3 commits intomasterfrom
AddUnitTesting
Mar 27, 2025
Merged

Add passing-status-required unit tests#9
dommmin-ic merged 3 commits intomasterfrom
AddUnitTesting

Conversation

@matthewsullivan-instacart
Copy link
Copy Markdown
Contributor

Instacart ticket INFRASEC-1353

Add unit tests, require passing status for merge.

@isc-bot
Copy link
Copy Markdown

isc-bot Bot commented Mar 21, 2025

You can type a comment on the PR to rebuild, refresh code freeze or other common operations. See here for more detail.
Here's a helpful guide to debug build failures.

Show ISC commands
Usage: isc [OPTIONS] COMMAND [ARGS]...

Options:
  -h, --help  Show this message and exit.

Commands:
  approve   [EMERGENCY USE ONLY] Approve a PR when no other reviewers are
            available.
  build     Trigger a build of this PR.
  describe  Trigger an AI description of this PR.
  help      Show the help text.
  launch    Launch the head of this PR to a non-production service.
  refresh   Refresh all statuses on this PR.
  review    Trigger an AI review of this PR.
  unblock   [EMERGENCY USE ONLY] Bypass build and test failures.
  unfreeze  Refresh the code freeze status on this PR.

@dommmin-ic dommmin-ic merged commit c439288 into master Mar 27, 2025
1 check passed
@dommmin-ic dommmin-ic deleted the AddUnitTesting branch March 27, 2025 18:09
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.

2 participants