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

Use micromamba instead of miniconda to speed up tests #1120

Merged
merged 1 commit into from
Nov 13, 2023

Conversation

haasad
Copy link
Member

@haasad haasad commented Nov 10, 2023

If we use micromamba instead of miniconda to run the tests in the github actions, we can bring down the time from ~20min to ~6min. This is still far from ideal, but at least the initial feedback is much faster with linux tests only taking ~2min.

relates: #1094 + #1097

Checklist

  • Keep pull requests small so they can be easily reviewed.
  • Categorize the PR by setting a good title and adding one of the labels:
    bug, feature, ui, change, documentation, breaking, ci
    as they show up in the changelog
  • Link this PR to related issues by using closing keywords.
  • Request a review from another developer.

@coveralls
Copy link

Coverage Status

coverage: 50.0% (-0.07%) from 50.066%
when pulling de49deb on micromamba-actions
into 7eebd42 on master.

@haasad haasad requested a review from marc-vdm November 11, 2023 00:05
@haasad haasad marked this pull request as ready for review November 11, 2023 00:09
@haasad haasad added the ci PRs related to CI of AB label Nov 11, 2023
@marc-vdm marc-vdm added this to the 2.9.3 milestone Nov 13, 2023
@marc-vdm marc-vdm merged commit f9da392 into master Nov 13, 2023
9 checks passed
@haasad haasad deleted the micromamba-actions branch December 8, 2023 06:18
Copy link

This issue has been implemented in the new release of Activity Browser 🚀 (version 2.9.3), you can get the new version by updating Activity Browser.

🤖beep boop! I'm a bot and this message was an automated action. If updating does not make sense for this issue, just ignore this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci PRs related to CI of AB
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants