Skip to content

Conversation

nsoranzo
Copy link
Member

@nsoranzo nsoranzo commented Sep 19, 2025

Equivalent to `--database_type postgres`.

Needed for Click 8.3.0, which reworked the handling of `flag_value` and
`default`:

https://click.palletsprojects.com/en/stable/changes/#version-8-3-0

Here the default (`None`) of `--postgres` overwrote the default (`auto`)
of `--database_type`, causing this error:

```
AssertionError: Planemo command [profile_create profilejobtest] resulted in unexpected exit code [1], expected exit code [0]]. Command output [] Exception [Unknown database type [None].]
```

The alternative solution of adding `default="auto"` to the `--postgres`
option would fail the assert at
https://github.com/galaxyproject/planemo/blob/ac2191df56b4de32cca239e3f257aa22f80b7ba0/planemo/context.py#L81
@nsoranzo nsoranzo force-pushed the enable_dependabot_gh_actions branch from ce2f1aa to fa45dc6 Compare September 20, 2025 18:55
@nsoranzo nsoranzo requested a review from mvdbeek September 20, 2025 19:06
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