Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix CLI install regressions #6214

Merged
merged 2 commits into from
Mar 19, 2024
Merged

Fix CLI install regressions #6214

merged 2 commits into from
Mar 19, 2024

Conversation

Alkarex
Copy link
Member

@Alkarex Alkarex commented Mar 19, 2024

  • It was not possible to pass a blank prefix
  • Docker crash when user already exists due to not returning proper exit code anymore. Regression from Command Line Parser Concept #6099

It was not possible to pass a blank prefix
@Alkarex Alkarex added CLI Command line interfaces in ./cli Bug (confirmed) 🐞 issues that are reproducable labels Mar 19, 2024
@Alkarex Alkarex added this to the 1.24.0 milestone Mar 19, 2024
The dedicated exit code was not sent anymore when a user already exists
@Alkarex Alkarex changed the title Fix CLI install with prefix Fix CLI install regressions Mar 19, 2024
@Alkarex Alkarex mentioned this pull request Mar 19, 2024
4 tasks
@Alkarex
Copy link
Member Author

Alkarex commented Mar 19, 2024

Merging a bit fast as it prevents some systems from working

@Alkarex Alkarex merged commit b5445e1 into FreshRSS:edge Mar 19, 2024
2 checks passed
@Alkarex Alkarex deleted the fix-cli-prefix branch March 19, 2024 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug (confirmed) 🐞 issues that are reproducable CLI Command line interfaces in ./cli
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant