docs: add Go and Jasmine reporter guides - #9
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2773375de0
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| --project ./services/api \ | ||
| --go-packages ./services/api/... |
There was a problem hiding this comment.
When the Go module is located at ./services/api, --project establishes that directory as the project root, so passing ./services/api/... again makes go test look for services/api/services/api and fail with “directory prefix … does not contain main module.” Go treats dot-prefixed package patterns as filesystem paths relative to the working directory (Go package-list documentation); use --go-packages ./... here or omit it because that is already the default.
Useful? React with 👍 / 👎.
GitHub Actions
Failure
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0acc9f9ca7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| jasmine.getEnv().addReporter( | ||
| new TestreamJasmineReporter({ | ||
| apiKey: process.env.TESTREAM_API_KEY, | ||
| uploadEnabled: process.env.TESTREAM_UPLOAD_ENABLED === "true", |
There was a problem hiding this comment.
When a user follows this basic configuration with only TESTREAM_API_KEY set, an unset TESTREAM_UPLOAD_ENABLED evaluates to false, overriding the reporter's documented true default and causing npx jasmine to write a local report without uploading it. Default this expression to enabled when the variable is absent, or explicitly instruct users to set the additional variable before the first run.
Useful? React with 👍 / 👎.
1 participant
Summary
static/llms.txtwith canonical docs URLs.Validation
npm run typechecknpm run buildnpm run test:indexnownpm run indexnow:dry-run -- --sitemap build/sitemap.xmlgit diff origin/main...HEAD --checkThe production build completed successfully. It emitted only existing Webpack cache permission warnings from the local environment.