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

add miniconda file to gitignore #293

Merged
merged 1 commit into from
Nov 8, 2023
Merged

add miniconda file to gitignore #293

merged 1 commit into from
Nov 8, 2023

Conversation

selenabr
Copy link
Collaborator

@selenabr selenabr commented Nov 8, 2023

  • fixes #

Description (what might a Reviewer want to know)

  • what methods did you move somewhere else (not written by yourself and therefore might not be related to the PR topic, this is no excuse for bad code!)

PR checklist

  • main-branch has been merged into local branch to resolve conflicts
  • tests and linter have passed AFTER local merge
  • at least one other dev reviewed and approved
  • documentation
  • tests

@selenabr selenabr requested a review from lucatreide November 8, 2023 14:33
Copy link

github-actions bot commented Nov 8, 2023

Coverage report

The coverage rate went from 71.42% to 71.42% ➡️
The branch rate is 64%.

None of the new lines are part of the tested code. Therefore, there is no coverage data about them.

@selenabr selenabr merged commit 3398bbe into main Nov 8, 2023
1 check passed
@selenabr selenabr deleted the selena-test branch November 8, 2023 14:41
@hendraet
Copy link
Collaborator

hendraet commented Nov 8, 2023

It might be nice to make this a bit more general because there are different files if you are using MacOS or other architectures than x86

@selenabr selenabr restored the selena-test branch November 8, 2023 17:19
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