chore(queueserver): remove fork boilerplate from the service top level - #123
Open
Anthony Sligar (sligara7) wants to merge 1 commit into
Open
chore(queueserver): remove fork boilerplate from the service top level#123Anthony Sligar (sligara7) wants to merge 1 commit into
Anthony Sligar (sligara7) wants to merge 1 commit into
Conversation
- Fold fork-era AUTHORS.rst/CONTRIBUTING.rst into PARITY.md ancestry credits (both were cookiecutter boilerplate; attribution now points at the upstream contributor records, license text stays in LICENSE) - Move store_test_durations.sh into scripts/ and update the two references (queueserver-tests workflow comment, docs contributing page) - docker-configs/ kept: referenced by README and scripts/start_LDAP.sh
There was a problem hiding this comment.
Pull request overview
Cleans up the backend/queueserver_service top-level by removing inherited cookiecutter boilerplate, preserving required operational assets, and relocating a CI calibration helper script while keeping references accurate.
Changes:
- Removed fork-inherited
AUTHORS.rstandCONTRIBUTING.rst, and added an upstream attribution note toPARITY.md. - Moved the test-duration calibration helper to
scripts/store_test_durations.shand updated references in docs and CI workflow comments. - Updated contributing documentation to point to the new script location.
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| backend/queueserver_service/scripts/store_test_durations.sh | Adds relocated helper script for pytest-split duration calibration. |
| backend/queueserver_service/PARITY.md | Adds upstream contributor credit note replacing removed boilerplate files. |
| backend/queueserver_service/docs/source/contributing.rst | Updates calibration instructions to reference the script’s new location. |
| backend/queueserver_service/CONTRIBUTING.rst | Removes stale cookiecutter contributing boilerplate. |
| backend/queueserver_service/AUTHORS.rst | Removes empty/stub authors boilerplate. |
| .github/workflows/queueserver-tests.yml | Updates workflow comment to reference the script’s new location. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
+141
to
142
| Calibration is performed by running the script ``scripts/store_test_durations.sh`` locally, | ||
| which saves execution time for each test in the ```.test_durations`` file. The file then |
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.
Top-level cleanup of backend/queueserver_service so the directory reads as one service:
No code, dependency, or shim changes; packaging verified by building and importing the wheel from the tree under the frozen lock.