Skip to content

update notification queue name#2155

Merged
seanaye merged 1 commit intomainfrom
seanaye/chore/dev-queue
Mar 24, 2026
Merged

update notification queue name#2155
seanaye merged 1 commit intomainfrom
seanaye/chore/dev-queue

Conversation

@seanaye
Copy link
Copy Markdown
Contributor

@seanaye seanaye commented Mar 24, 2026

The sops file had the wrong value for the ingress queue so it was pushing into the egress queue rather than the ingress queue

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 24, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 3dab84b7-cad3-48a1-af76-42177f2a7514

📥 Commits

Reviewing files that changed from the base of the PR and between a2487ab and c54bb8f.

📒 Files selected for processing (1)
  • .env-localdev.enc

Walkthrough

The .env-localdev.enc encrypted configuration file was updated with a new encrypted value for the NOTIFICATION_QUEUE environment variable. The change also includes refreshed SOPS metadata: the sops_lastmodified timestamp was updated from 2026-03-19T16:10:48Z to 2026-03-24T20:27:59Z, and the sops_mac checksum was regenerated with a new encrypted value. Three lines were modified in total with no changes to public entity declarations.

Poem

🐰 A secret was whispered, then locked up so tight,
The queue got refreshed in the pale morning light,
SOPS checked the lock, the timestamp renewed,
The rabbit hops happy, the MAC value's shrewd! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'update notification queue name' directly describes the main change in the PR—updating the encrypted NOTIFICATION_QUEUE value in the .env-localdev.enc file.
Description check ✅ Passed The description clearly explains the purpose of the change: correcting the sops file's notification queue value to push into the ingress queue instead of the egress queue, which aligns with the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch seanaye/chore/dev-queue

Comment @coderabbitai help to get the list of available commands and usage tips.

@seanaye seanaye merged commit f53e406 into main Mar 24, 2026
24 checks passed
@seanaye seanaye deleted the seanaye/chore/dev-queue branch March 24, 2026 20:41
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.

2 participants