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

ci: don't use cache to build with Autoware #153

Merged
merged 2 commits into from
Oct 10, 2023
Merged

Conversation

takeshi-iwanari
Copy link
Contributor

@takeshi-iwanari takeshi-iwanari commented Oct 9, 2023

Description

This PR fixes CI (Build with Autoware) failure by disabling cache for Docker.
Building Autoware failed even without CARET but a new PR fixes the error recently, but the new PR is not reflected in our CI due to cache. That's why CI kept failed.

Related links

None

Notes for reviewers

None

Pre-review checklist for the PR author

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

  • The PR has been properly tested.
  • The PR has been reviewed.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • (Optional) The PR has been properly tested with CARET_report verification.
  • There are no open discussions or they are tracked via tickets.
  • The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

@takeshi-iwanari takeshi-iwanari changed the title Ci dont use cache ci: dont use cache Oct 9, 2023
@takeshi-iwanari takeshi-iwanari changed the title ci: dont use cache ci: dontt use cache to build with Autoware Oct 9, 2023
@takeshi-iwanari takeshi-iwanari changed the title ci: dontt use cache to build with Autoware ci: don't use cache to build with Autoware Oct 9, 2023
@takeshi-iwanari
Copy link
Contributor Author

@nabetetsu
Could you review this PR?

@takeshi-iwanari takeshi-iwanari merged commit 70c1cce into main Oct 10, 2023
16 checks passed
@takeshi-iwanari takeshi-iwanari deleted the ci_dont_use_cache branch October 10, 2023 06:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants