Skip to content

fix(CI): Install composer dependencies of Talk - #36389

Merged
nickvergessen merged 1 commit into
masterfrom
nickvergessen-patch-1
Jan 27, 2023
Merged

fix(CI): Install composer dependencies of Talk#36389
nickvergessen merged 1 commit into
masterfrom
nickvergessen-patch-1

Conversation

@nickvergessen

Copy link
Copy Markdown
Member
  • Resolves: N/A

Summary

Ref nextcloud/spreed#8578
Found in #36310 (comment)

Checklist

@ChristophWurstChristophWurst 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.

🙏

@danxuliudanxuliu 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.

CI is not happy:

+ composer install --no-dev
/bin/sh: composer: not found

Comment thread.drone.yml
Signed-off-by: Joas Schilling <213943+nickvergessen@users.noreply.github.com>
@ChristophWurst

Copy link
Copy Markdown
Member

you can also use ci as prefix for conventional commits. tools that generate changelogs would then hide the commit. a fix would be listed as a fix on the product. end of nitpick.

@nickvergessen

Copy link
Copy Markdown
MemberAuthor

So, we are green and good again :)

@nickvergessen
nickvergessen merged commit af5910b into masterJan 27, 2023
@nickvergessen
nickvergessen deleted the nickvergessen-patch-1 branch January 27, 2023 09:29
@kesselbkesselb mentioned this pull request Jan 27, 2023
9 tasks
@blizzzblizzz mentioned this pull request Feb 1, 2023
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@nickvergessen@ChristophWurst@vitormattos@blizzz@danxuliu