Skip to content

Remove deprecated occ app:check-code command#32210

Merged
come-nc merged 1 commit into
masterfrom
fix/remove-deprecated-occ-app-check-code-command
Apr 28, 2022
Merged

Remove deprecated occ app:check-code command#32210
come-nc merged 1 commit into
masterfrom
fix/remove-deprecated-occ-app-check-code-command

Conversation

@come-nc

@come-nc come-nc commented Apr 28, 2022

Copy link
Copy Markdown
Contributor

The command does nothing and outputs:

The app code checker doesn\t check anything and this command will be removed in Nextcloud 23

Signed-off-by: Côme Chilliet come.chilliet@nextcloud.com

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc come-nc added the 3. to review Waiting for reviews label Apr 28, 2022
@come-nc come-nc added this to the Nextcloud 25 milestone Apr 28, 2022
@come-nc come-nc self-assigned this Apr 28, 2022
@come-nc come-nc requested review from a team, ArtificialOwl, CarlSchwan and juliusknorr and removed request for a team April 28, 2022 13:10

@nickvergessen nickvergessen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥

@come-nc

come-nc commented Apr 28, 2022

Copy link
Copy Markdown
Contributor Author

Drone failure unrelated

@come-nc come-nc merged commit 1227278 into master Apr 28, 2022
@come-nc come-nc deleted the fix/remove-deprecated-occ-app-check-code-command branch April 28, 2022 14:44
@ajs124 ajs124 mentioned this pull request Oct 23, 2022
13 tasks
@ChristophWurst

Copy link
Copy Markdown
Member

The command does nothing and outputs

#25577 well intended. I just forgot to remove the command later on :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants