Skip to content

fix: close all xfails, resolve review findings, remove prek overlay#1

Open
wgordon17 wants to merge 1 commit intogordon-code:mainfrom
wgordon17:fix/close-xfails-and-review-findings
Open

fix: close all xfails, resolve review findings, remove prek overlay#1
wgordon17 wants to merge 1 commit intogordon-code:mainfrom
wgordon17:fix/close-xfails-and-review-findings

Conversation

@wgordon17
Copy link
Member

Summary

  • Close all 5 xfail tests (prek lint, cog verify, copier update)
  • Redesign install recipe: commit before hooks, fix cog scope/nix fmt bugs
  • Remove prek binary-fetch overlay (nixpkgs ships v0.3.4)
  • Vendor cog-schema.json, disable CI telemetry, conditional README quickstart
  • Add just update recipe, per-platform hosting_org validator, GR5 Renovate fix
  • Migrate ~18 tests to session fixtures, rewrite derivation test

67 tests pass, 0 xfail. Behavioral integration test passes.

- Close 5 xfail tests: prek lint (TAB indent, trailing newline, branch checkout), cog verify (conventional commits), copier update (--answers-file)

- Redesign install recipe: commit before hooks, fix cog scope syntax

- Remove 44-line prek binary-fetch overlay (nixpkgs is v0.3.4)

- Vendor cog-schema.json locally, disable CI telemetry

- Conditional README quickstart via includes/ fragment

- Add just update recipe, tighten hosting_org validator per-platform

- Extend Renovate fileMatch to includes/*.jinja (GR5)

- Fix nix fmt stdin, migrate tests to session fixtures, rewrite derivation test
@wgordon17 wgordon17 force-pushed the fix/close-xfails-and-review-findings branch from fbea2c4 to 092f58e Compare March 13, 2026 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant