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): occ error handling #595

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

joshtrichards
Copy link
Member

Closes #260

CLI mode:

  • add error handling for occ upgrade
  • fix broken error handling for occ maintenance:mode
  • add logging when these are skipped)
  • refactor some of the wording for clarity/consistency

Signed-off-by: Josh Richards <[email protected]>
@joshtrichards joshtrichards added bug 3. to review feature: upgrade Note: In `server` but called by updater. Also see `feature: installs and updates` in server mode: cli labels Dec 24, 2024
@joshtrichards
Copy link
Member Author

Test failure in master is unrelated. It's due to nextcloud/server#49224 and currently impacting all PRs in this repo.

@joshtrichards joshtrichards added this to the Nextcloud 31 milestone Dec 24, 2024
@nickvergessen
Copy link
Member

Test failure in master is unrelated. It's due to nextcloud/server#49224 and currently impacting all PRs in this repo.

I will take care of that.

But the "check-same-code-base" is related. You need to regenerate the phar

@nickvergessen
Copy link
Member

Daily generating script was adjusted. I think it's automatically deployed, but we will see tomorrow I guess

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review bug feature: upgrade Note: In `server` but called by updater. Also see `feature: installs and updates` in server mode: cli
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Permission denied for 'occ'
2 participants