Vendor the launch-monitor articles from AffineDrift; remove the local copies - #34
Merged
Merged
Conversation
… copies
Both golf articles previously maintained here -- the LaTeX technology review
under `tech-review/` and its companion `notes/screw-theory-research-outline.md`
-- now live in AffineDrift, so all the golf content shares one repository, one
CI, and one release path. This repo reaches them through a submodule,
`vendor/affinedrift`, the same way the fleet's other repositories (e.g.
Gasification_Model, UpstreamDrift) vendor `D-sorganization/Tools` at
`vendor/ud-tools`.
Pinned to 590fb98, the first commit on AffineDrift main where the review
builds cleanly end to end -- verified there by a full local pdflatex rebuild
(0 errors, 62 pages) before pinning here, not assumed from the migration PRs
having merged.
### Removed
* `tech-review/` -- the LaTeX source, bibliography, and eight research
dossiers;
* `notes/screw-theory-research-outline.md` -- the outline is not re-edited
before deletion, since it now exists only in the submodule and the
version there is authoritative;
* `.github/workflows/tech-review.yml` -- the document is built by
AffineDrift's `compile-textbooks.yml` now, alongside its other LaTeX
documents;
* the `.gitignore` entries for `tech-review/*` build artifacts -- those
are AffineDrift's concern now.
### Updated, not just removed
`README.md` no longer points readers at a directory that does not exist:
the "one piece written to be read by others" line and the directory map are
rewritten to point at the submodule, and the license note is corrected to
say the review's license status travels with it rather than restating
`tech-review/`-specific wording for a path that is gone.
Co-Authored-By: Claude Fable 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.
Both golf articles previously maintained here now live in AffineDrift, so all the golf content shares one repository, one CI, and one release path. This repo reaches them through a submodule,
vendor/affinedrift— the same way the fleet's other repositories (e.g.Gasification_Model,UpstreamDrift) vendorD-sorganization/Toolsatvendor/ud-tools.Pinned to
590fb98, the first commit on AffineDrift main where the review builds cleanly end to end — verified there with a full localpdflatexrebuild (0 errors, 62 pages) before pinning here, not assumed from the migration PRs having merged.Removed
tech-review/— the LaTeX source, bibliography, and eight research dossiersnotes/screw-theory-research-outline.md— not re-edited before deletion; it now exists only in the submodule and that copy is authoritative.github/workflows/tech-review.yml— the document is built by AffineDrift'scompile-textbooks.ymlnow, alongside its other LaTeX documents.gitignoreentries fortech-review/*build artifacts — AffineDrift's concern nowUpdated, not just removed
README.mdno longer points readers at a directory that doesn't exist: the "one piece written to be read by others" line and the directory map now point at the submodule, and the license note is corrected to say the review's license status travels with it rather than restatingtech-review/-specific wording for a path that's gone.Checked before removing anything
vendor/and the removed directory itself) was searched for references to the old paths — the only survivor is the deliberate historical note in the new README section, "previouslytech-review/and ... — now ...".quality-gate.yml,local-only-runner-guard.yml) don't referencetech-reviewand don't depend on it.160000), not an embedded copy of the files.Verified