FlexMeasures / flexmeasures / 31213094017
84%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 176
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 84.112%. Remained the same
31213094017

push

github

web-flow 
Make clean-db work on macOS, and let poe actually pass it arguments and prompts (#2408)

* data/scripts: make clean_database.sh work on macOS as well

Context:
- Issue #1280: the script only ran on Linux distributions.
- It reached the server through "sudo -i -u postgres", but Homebrew and
  Postgres.app create no "postgres" system account, so on macOS every call
  failed with "sudo: unknown user postgres".
- Those installs instead make the installing user a superuser.

Change:
- Detect the platform once and route psql/createdb/dropdb through a
  superuser prefix: "sudo -i -u postgres" (plus "-U postgres") on Linux,
  and nothing at all on macOS.
- Name the database on every psql call, since psql would otherwise fall
  back to a database named after the current user, which macOS installs
  do have but Linux ones do not. The privilege grant consequently now
  applies to the new database rather than to the maintenance database.
- Check up front that the server is reachable, and say how to start it,
  instead of letting a stack of psql errors scroll by.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

* docs: note macOS support for clean-db in the changelog

Context:
- Issue #1280 made "uv run poe clean-db" unusable on macOS.

Change:
- Added an Infrastructure / Support entry for the fix.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

* pyproject: let poe clean-db reach the script with its arguments and prompts

Context:
- "poe clean-db --db-name x" always failed with "db-name is required".
- Poe only exposes task args as environment variables, so the script,
  which reads $1 and $2, received nothing at all.
- The task was also a "shell" task, and poe feeds a shell task's content
  to the interpreter over stdin. That left the script with no terminal to
  read from, so its password and confirmation prompts read empty: a new
  user got an empty password, and an existing database was silently left
  alone because the "recreate?" prompt defau... (continued)

17312 of 20582 relevant lines covered (84.11%)

0.84 hits per line

Jobs
ID Job ID Ran Files Coverage
1 31213094017.1 176
84.11
 GitHub Action Run
Source Files on build 31213094017