Repository navigation
fix(conventions): judge only stack ports in DEVC-21; date to 0.9.1 - #115
Merged
Merged
Conversation
Release-please cut #113 as 0.9.1 (a feat is a patch before 1.0.0), so the eleven requirements it added are since 0.9.1, not 0.10.0. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
An application's own forwarded port, such as platform-api's API dev server on 17200, is not a stack port and is the application's to choose. DEVC-21 now judges a forwarded port only when a stack publishes it or listens on it, or when it falls in the 154xx band. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Two corrections to #113, which shipped in 0.9.1.
.claude/rules/port-reservations.md. DEVC-21 now judges a forwarded port only when it belongs to the stacks, or falls in the 154xx band:5432still fires;An application's own port is the application's to choose. The title, rationale and examples say so, and the near-miss fixture and a new Rego test forward 17200.
featas a patch, so feat(conventions): move the scaffold's general rules here #113 shipped as 0.9.1. The eleven requirements it added saidsince: 0.10.0, a version that does not exist. Nothing checkssince.Test plan
checks:test(1078 passed),checks:lint,checks:checktests/test_invariants.py🤖 Generated with Claude Code