Skip to content

Document the lessons from the ci-setup trials - #10

Merged
PavlosIsaris merged 1 commit into
mainfrom
trial-lessons
Oct 1, 2026
Merged

PavlosIsaris merged 1 commit into
mainfrom
trial-lessons

Conversation

@PavlosIsaris

Copy link
Copy Markdown
Contributor

Lessons from two fresh-agent tests and the full ci-setup trial on volunteering-matching-platform (its PR #29):

  • Evidence rule. Every input that an agent or a person sets needs a file and a line in the repository. A fresh agent copied the browser-test recipe into a repository with no browser suite.
  • Database. The guide now says what to do when the tests need MySQL or PostgreSQL: move the tests to SQLite in memory, or keep the old CI. It also explains why key:generate fails when the application reads the database while it boots.
  • Recipe. The browser-test recipe is for repositories that have a browser suite only. It also warns when composer check already runs check:types.
  • Triggers. The templates do not run on pushes to every branch.
  • Pitfalls. New rows for a test database and for a missing storage/framework/views folder ("Please provide a valid cache path.").

AGENTS.md has the same evidence and database rules. Release plan after merge: v0.1.6, documentation only.

- Every input an agent or a person sets needs evidence in the repository;
  do not copy a recipe without it
- What to do when the tests need MySQL or PostgreSQL, and why an
  application that reads the database while it boots fails every job
- The browser-test recipe needs a browser suite; composer check must not
  already run check:types
- The templates do not run on pushes to every branch
- A missing storage/framework/views folder fails only in CI
@PavlosIsaris
PavlosIsaris merged commit 67fa099 into main Oct 1, 2026
28 checks passed
@PavlosIsaris
PavlosIsaris deleted the trial-lessons branch October 1, 2026 06:45
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