docs: remove redundant SWR/SPEC for Betterstack feature [PYSDK-127]#635
Open
olivermeyer wants to merge 1 commit into
Open
docs: remove redundant SWR/SPEC for Betterstack feature [PYSDK-127]#635olivermeyer wants to merge 1 commit into
olivermeyer wants to merge 1 commit into
Conversation
The user-facing contract was already covered by existing artefacts: SWR-SYSTEM-GUI-HEALTH-1 authorises health visibility in the Launchpad footer; SPEC_GUI_SERVICE FR-06 covers the embedded status indicator; SPEC_PLATFORM_SERVICE FR-04 covers per-environment endpoint config, and §7.2 covers URL validation. The per-environment Betterstack URL resolution is a design refinement, not a new design input. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
olivermeyer
requested review from
a team and
helmut-hoffer-von-ankershoffen
as code owners
|
There was a problem hiding this comment.
Pull request overview
This PR primarily removes redundant QMS documentation artifacts (SWR/SPEC) related to the per-environment Betterstack status-page work, but it also includes a repo-wide version bump to 1.4.0 and a change to an application CLI test.
Changes:
- Delete
requirements/SWR_SYSTEM_GUI_STATUS_PAGE_1.mdandspecifications/SPEC-LAUNCHPAD-STATUS-PAGE.md. - Bump project version references from
1.3.0to1.4.0across release/version artifacts. - Update
test_cli_application_dump_schematato pass--application-versionexplicitly and adjust the expected zip filename (and change its pytest marking).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
VERSION |
Bump SDK version to 1.4.0. |
uv.lock |
Update editable package version to 1.4.0. |
pyproject.toml |
Update project version and bumpversion current_version to 1.4.0. |
sonar-project.properties |
Update Sonar project version to 1.4.0. |
docs/source/conf.py |
Update Sphinx docs version to 1.4.0. |
CLI_REFERENCE.md |
Update displayed CLI version string to 1.4.0. |
examples/notebook.py |
Update example pin to aignostics==1.4.0. |
aignostics.spec |
Update app bundle version to 1.4.0. |
tests/aignostics/application/cli_test.py |
Make schemata dump test deterministic via --application-version and change its pytest marker usage. |
requirements/SWR_SYSTEM_GUI_STATUS_PAGE_1.md |
Delete redundant SWR artifact. |
specifications/SPEC-LAUNCHPAD-STATUS-PAGE.md |
Delete redundant SPEC artifact. |
Codecov Report✅ All modified and coverable lines are covered by tests. |
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.
Why?
PR #599 added
requirements/SWR_SYSTEM_GUI_STATUS_PAGE_1.mdandspecifications/SPEC-LAUNCHPAD-STATUS-PAGE.mdto cover the per-environment Betterstack status-page work, but the user-facing contract was already fully authorised by existing artefacts —SWR-SYSTEM-GUI-HEALTH-1(health visibility in the Launchpad footer),SPEC_GUI_SERVICEFR-06 (embedded health-monitoring integration in the navigation frame), andSPEC_PLATFORM_SERVICEFR-04 + §7.2 (per-environment endpoint configuration and URL validation). The per-environment URL resolution is a SPEC-level design refinement, not a new design input, and the duplicate artefacts violate the/software-specificationsconciseness rule (S13) and create approval churn for no contract change. Tracked under the existing CR PYSDK-127.How?
Delete both files; make no edits to any other SWR or SPEC. The trace chain
SHR-SYSTEM-1 → SWR-SYSTEM-GUI-HEALTH-1 → SPEC_GUI_SERVICE / SPEC_PLATFORM_SERVICE → code/testsremains intact and unchanged.