chore: untrack two committed store files, add the rules that let them in - #364
Open
ywatanabe1989 wants to merge 1 commit into
Open
chore: untrack two committed store files, add the rules that let them in#364ywatanabe1989 wants to merge 1 commit into
ywatanabe1989 wants to merge 1 commit into
Conversation
Two database files were tracked in this repo: a 68 KB clew runtime store at .scitex/clew/runtime/db.sqlite, and a second one under a legacy clew example. Both are binary, so `git grep -I` skips them and neither appears in any text scan of this tree - the only thing that surfaced them was a binary-inclusive sweep. They are untracked here, and the .gitignore gains the patterns that were missing: the existing rules covered only Django's db.sqlite3, which is why these two slipped in. Three stale sentences are corrected in the same pass. All three describe clew's provenance store as a local file database; clew moved its stores onto the per-host PostgreSQL and has no database file, so the wording sent a reader looking for something that is never written: docs/sphinx/core_concepts.rst (twice) and docs/sphinx/modules/io.rst. pyproject's note on the `clew` extra claimed it pulls in nothing and uses only stdlib. It pins scitex-clew==0.17.0, and clew owns its own store; the note now says that. The .env example's comment describing CROSSREF_LOCAL_DB is made generic - it names a path, not a format. One entry is dropped from tests/integration/test_cross_package_imports.py: scitex_io._load_modules._sqlite3 no longer exists on scitex-io's develop, so that parametrised case was doing nothing but SKIPPING, every run, with "auto-gen snapshot drift". Measured: 60 skips before, 59 after, same 254 passed. The block is auto-generated and the generator (`ecosystem write-integration-tests`) is not in the installed scitex-dev, so it is corrected by hand; a regeneration would drop it too, since nothing in this tree references the module.
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.
The finding this PR exists for. Two database files were tracked in this repo — a 68 KB clew runtime store at
.scitex/clew/runtime/db.sqlite, and a second under a legacy clew example. Both are binary, sogit grep -Iskips them and neither appears in any text scan of this tree; only a binary-inclusivegit grep -asurfaced them. They are untracked here, and.gitignoregains the patterns that were missing — the existing rules covered only Django'sdb.sqlite3, which is exactly why these two slipped in.Three stale sentences corrected. All three described clew's provenance store as a local file database. scitex-clew moved its stores onto the per-host PostgreSQL and has no database file, so the wording sent a reader looking for something never written:
docs/sphinx/core_concepts.rst(×2) anddocs/sphinx/modules/io.rst.pyproject.tomlclaimed theclewextra pulls in nothing and uses only stdlib. It pinsscitex-clew==0.17.0, and clew owns its own store.One dead test entry removed.
scitex_io._load_modules._sqlite3no longer exists on scitex-io'sdevelop, so that parametrised case was doing nothing but skipping, every run, with "auto-gen snapshot drift". Measured with scitex-python's own venv: 60 skips before, 59 after, 254 passed both times, 2 xfailed — exactly one silent skip removed, none created. The block is auto-generated and its generator (ecosystem write-integration-tests) is not in the installed scitex-dev, so it is corrected by hand; a regeneration would drop it too, since nothing in this tree references the module.Before / after (excluding
examples/_legacy/, as measured): 17 lines → 13, of which the ignore-rule count rose 4 → 6 on purpose. Non-ignore-rule: 13 → 7. What stays, and why:README.md×3,docs/05_ADDITIONAL_MODULES.md×1stx.db.SQLite3, a public class that exists in thescitex-dbpackage. scitex-db is mid-cutover (it shipsdocs/sqlite-to-postgres-cutover.mdand a_migrate/surface). Deleting the docs while the class ships would make this repo misdescribe its own umbrella. scitex-python cannot reach zero until scitex-db retires that class.tests/integration/test_integration.py:301assert hasattr(stx.db, "SQLite3")is coverage of that same class. Removing it deletes a check, not a dependency.scripts/maintenance/_pypi_packages.py:153import sqlite3as a missing package. A detector must name what it recognises.docs/guides/CROSSREF_API_CONFIGURATION.md:149- **Format**: …describes the on-disk shape of an existing 1.2 TB / 167 M-row CrossRef mirror. Rewriting it makes the doc lie about what is on the NAS. Revisit when that mirror moves..gitignore×6examples/_legacy/is left alone (31 further lines, legacy notebooks demonstrating the same livescitex-dbAPI).Positive control (
stx.db) returns 4 files on the same invocation, so the reduced counts are real absences.