Skip to content

Main into staging (release 215)#1477

Merged
martha merged 8 commits into
stagingfrom
main
Jun 4, 2026
Merged

Main into staging (release 215)#1477
martha merged 8 commits into
stagingfrom
main

Conversation

@martha

@martha martha commented May 26, 2026

Copy link
Copy Markdown
Contributor

Description

Summary of changes:

Depends on hmis-warehouse PR:

How to test:

Type of change

Checklist before requesting review

  • I have performed a self-review of my code
  • I have run the code that is being changed under ideal conditions, and it doesn't fail
  • I have used Axe DevTools to scan for accessibility issues (or not applicable)
  • My code includes comments and/or descriptive variable names to help other engineers understand the intent (or not applicable)
  • I have updated the documentation (or not applicable)
  • If it's not obvious how to test this change, I have provided testing instructions in this PR or the related issue

martha and others added 7 commits May 22, 2026 07:47
* Simplify Assessment delete perms checks

* Use new permission field canDelete

* resolve canDeleteAssessment instead of canDelete
* uuid

* Minor upgrade transitive deps
* Upgrade Docker base images from Alpine to Debian bookworm-slim

Aligns with other GR repos (e.g. hmis-warehouse uses slim-bookworm).
Also fixes a COPY --chown ordering bug where the app user didn't exist
yet when Docker resolved the chown, and adds --chown=app:app to all
COPY commands targeting /etc/nginx.

* Update docker/Dockerfile

Co-authored-by: Theron Toomey <ttoomey@greenriver.org>

* Add ca-certificates to Docker builder stage

bookworm-slim omits ca-certificates by default; without it the Sentry
CLI native binary cannot verify SSL when uploading source maps during
yarn build, causing CI to fail.

* Upgrade nginx image from 1.28 to 1.30

* Switch Docker base images from bookworm to trixie

nginx 1.30 is not published for Debian 12 (bookworm); trixie (Debian 13)
is required. Updated test-backend Ruby image to trixie for consistency.

* Fix Dockerfile linter warnings

Update ENV instructions to key=value format (LegacyKeyValueFormat).
Suppress unfixable warnings with # check=skip pragma and comments
explaining why each suppression is intentional.

---------

Co-authored-by: Theron Toomey <ttoomey@greenriver.org>
@martha martha marked this pull request as ready for review June 4, 2026 17:10
@martha martha merged commit 8db43c7 into staging Jun 4, 2026
1 check passed
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.

3 participants