go version#2738
Open
realler13 wants to merge 35 commits into
Open
Conversation
ci.yaml commit
go version change for yml
updated CI with tests
fixed the broken pipe
test coverage added
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Add tests and CI status badge
formatting check
linting check now
now linting should pass
gosec now
- G112: add ReadHeaderTimeout to http.Server (Slowloris guard) - G706: validate PORT is numeric and log the int, not the tainted string - G104: handle the w.Write error in respondWithJSON Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Fix gosec findings: server timeout, log injection, unhandled write
It's a machine-specific permission allowlist that grows per session and was blocking branch switches. Gitignore it and untrack it; the working copy stays on disk. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Stop tracking .claude/settings.json (local tooling config)
Runs scripts/buildprod.sh to cross-compile the linux/amd64 binary. Fixed a typo (Deploly->Deploy) and the misleading job key (tests->deploy). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Add cd workflow to build the app on push to main
Staging spot for GCP service-account keys etc. — never committed. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Authenticates via GCP_CREDENTIALS, sets up Cloud SDK, and runs gcloud builds submit to build and push the image to Artifact Registry. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Add gcloud auth + build/push steps to cd workflow
- Add 'Deploy to Cloud Run' step (allow-unauthenticated, max-instances=4) - Bump auth/setup-gcloud to @V3 (Node 24 runtime; clears deprecation warning) - Change homepage heading to "Welcome to Notely" Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Deploy to Cloud Run + bump google actions to v3
- Add goose install + migrate step (goose turso ... up before deploy) - Job-level DATABASE_URL env for the migrate step - Pass DATABASE_URL to the Cloud Run service via --update-env-vars so the live app connects to the database (CRUD works) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Wire up DB migrations and DATABASE_URL in CD
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
1 participant
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.
No description provided.