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: remove the use of our container in check-hashes job #292

Closed
wants to merge 4 commits into from

Conversation

dehanj
Copy link
Member

@dehanj dehanj commented Nov 12, 2024

The container is not needed for that simple job, but it takes quite some time to set it up for each run.

Description

Please include a summary of the changes made in this PR and provide any context necessary for the change.

Fixes # (issues)

Type of change

Please tick any that are relevant to this PR and remove any that aren't.

  • Bugfix (non breaking change which resolve an issue)
  • Feature (non breaking change which adds functionality)
  • Breaking Change (a change which would cause existing functionality to not work as expected)
  • Documentation (a change to documentation)

Submission checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my changes
  • I have tested and verified my changes on target
  • My changes are well written and CI is passing
  • I have squashed my work to relevant commits and rebased on main for linear history
  • I have added a "Co-authored-by: x" if several people contributed, either pair programming or by squashing commits from different authors.
  • I have updated the documentation where relevant (readme, dev.tillitis.se etc.)
  • QEMU is updated to reflect changes

The container is not needed for that simple job, but it takes quite some
time to set it up for each run.
@dehanj
Copy link
Member Author

dehanj commented Nov 12, 2024

This is apparently not possible: https://github.com/actions/cache?tab=readme-ov-file#cache-version

@dehanj dehanj closed this Nov 12, 2024
@dehanj dehanj deleted the speed_up_ci branch November 12, 2024 15:27
@dehanj dehanj restored the speed_up_ci branch November 13, 2024 08:02
@dehanj dehanj reopened this Nov 13, 2024
@dehanj
Copy link
Member Author

dehanj commented Nov 13, 2024

Tried i a few more options with enableCrossOsArchive, it did not work. Giving up for now.

@dehanj dehanj closed this Nov 13, 2024
@dehanj dehanj deleted the speed_up_ci branch November 22, 2024 14:11
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.

1 participant