Skip to content

feat: accept getting-alt-ips status#802

Open
alexey-yarmosh wants to merge 1 commit intomasterfrom
block-to-probe-ips
Open

feat: accept getting-alt-ips status#802
alexey-yarmosh wants to merge 1 commit intomasterfrom
block-to-probe-ips

Conversation

@alexey-yarmosh
Copy link
Member

@coderabbitai
Copy link

coderabbitai bot commented Feb 19, 2026

No actionable comments were generated in the recent review. 🎉


Walkthrough

This pull request adds a new status value 'getting-alt-ips' to the Probe status definitions. The change is implemented in two parallel locations: the schema validation file updates the statusSchema allowed values, and the types file updates the Probe status union type to include the new value. Both modifications are minimal, consisting of a single line addition and removal each.

Suggested reviewers

  • MartinKolarik
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: adding support for a new 'getting-alt-ips' status value to the system.
Description check ✅ Passed The description references a related issue and is directly connected to the changeset's purpose of adding the 'getting-alt-ips' status.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch block-to-probe-ips

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Comment @coderabbitai help to get the list of available commands and usage tips.

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

Comments