Skip to content

Small fixes to action server port parsing, github workflows#1827

Merged
dandelany merged 2 commits intodevelopfrom
dev/small-fixes
May 6, 2026
Merged

Small fixes to action server port parsing, github workflows#1827
dandelany merged 2 commits intodevelopfrom
dev/small-fixes

Conversation

@dandelany
Copy link
Copy Markdown
Collaborator

Sweeping up a couple tiny fixes I had in my notes:

  • Action server was doing parseInt(AERIE_DB_PORT, 5432), like it was a fallback, in fact second argument is radix and this always returns NaN. This caused it to fallback to a default (5432) in another layer and never use custom AERIE_DB_PORT if passed
  • Publish workflow should require init step since it uses output from it on line 144. Also updated to make security scan scan the images from the PR branch instead of develop
  • Pgcmp workflow had wrong paths for scripts, this only matters for auto-running the workflow if/when any of those scripts change

Comment thread .github/workflows/publish.yml
Comment thread action-server/src/threads/worker.ts Outdated
Copy link
Copy Markdown
Contributor

@Mythicaeda Mythicaeda left a comment

Choose a reason for hiding this comment

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

Approving as neither of my comments are blocking

@dandelany dandelany merged commit c8dab71 into develop May 6, 2026
17 checks passed
@dandelany dandelany deleted the dev/small-fixes branch May 6, 2026 23:31
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