Skip to content

[Stable10] imrpove drone cache#34717

Closed
patrickjahns wants to merge 3 commits into
stable10from
stable10_imrpove_drone_cache
Closed

[Stable10] imrpove drone cache#34717
patrickjahns wants to merge 3 commits into
stable10from
stable10_imrpove_drone_cache

Conversation

@patrickjahns

@patrickjahns patrickjahns commented Mar 8, 2019

Copy link
Copy Markdown
Contributor

Backport of #34700

I've already purged the cache on minio - so this will come in clean

Signed-off-by: Patrick Jahns <github@patrickjahns.de>
Signed-off-by: Patrick Jahns <github@patrickjahns.de>
@patrickjahns patrickjahns added this to the development milestone Mar 8, 2019
@patrickjahns patrickjahns self-assigned this Mar 8, 2019

@phil-davis phil-davis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Note: 2 lines less diff because the PR in master had some extra blank lines on the end of .gitignore

@patrickjahns

Copy link
Copy Markdown
Contributor Author

Fails now on make vendor-bin as some iterations still use php5.6 and the tools don't have that.

I wanted to avoid to sprinkel all steps with potential composer pulls to have:

    environment:
      - COMPOSER_HOME=/drone/src/.cache/composer

Additionally it would have introduced complexity for building the cache in a single iteration

@phil-davis

Copy link
Copy Markdown
Contributor

commits have been cherry-picked into PR #34698 and that has been merged.

@phil-davis phil-davis closed this Mar 13, 2019
@phil-davis
phil-davis deleted the stable10_imrpove_drone_cache branch March 13, 2019 06:58
@lock lock Bot locked as resolved and limited conversation to collaborators Mar 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants