Skip to content
This repository has been archived by the owner on Jun 5, 2024. It is now read-only.

no dependabot in WFSim #358

Merged
merged 5 commits into from
Apr 5, 2022
Merged

no dependabot in WFSim #358

merged 5 commits into from
Apr 5, 2022

Conversation

JoranAngevaare
Copy link
Contributor

@JoranAngevaare JoranAngevaare commented Apr 1, 2022

What is the problem / what does the code in this PR do
As requested by @terliuk , maybe we can run dependabot somewhere else to avoid cluttering the PRs and receiving thousands of emails.

How does it work?
Check if the requirements are compatible with the software here instead:
https://github.com/XENONnT/ax_env

One added benefit is that we can also use the same package in strax, straxen, pema etc. avoiding doing all the dependencies multiple times

What will dependabot still do for WFSim?

  • Check updates for the docs (aren't many so we should be fine)
  • Check the version of strax and straxen
  • Check github actions changes, but there shouldn't be a lot - we can disable if needed later

PR family:

@coveralls
Copy link

coveralls commented Apr 1, 2022

Coverage Status

Coverage increased (+0.1%) to 81.459% when pulling 67142be on no_dependabot into 8c58d16 on master.

@JoranAngevaare JoranAngevaare merged commit 2093bb5 into master Apr 5, 2022
@ramirezdiego ramirezdiego deleted the no_dependabot branch April 6, 2022 12:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants