Skip to content

plumb context to DB queries#345

Merged
zoom-ua merged 3 commits intomasterfrom
joshblum/dbctx
Apr 27, 2026
Merged

plumb context to DB queries#345
zoom-ua merged 3 commits intomasterfrom
joshblum/dbctx

Conversation

@zoom-ua
Copy link
Copy Markdown
Contributor

@zoom-ua zoom-ua commented Apr 27, 2026

Comment thread githubbot/githubbot/db.go
func (d *DB) DeleteBranchesForRepo(convID chat1.ConvIDStr, repo string) error {
return d.RunTxn(func(tx *sql.Tx) error {
_, err := tx.Exec(`
DELETE FROM subscriptions
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

legit typo on the DB table

@zoom-ua zoom-ua merged commit 90c4090 into master Apr 27, 2026
2 checks passed
@zoom-ua zoom-ua deleted the joshblum/dbctx branch April 27, 2026 16:02
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