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.

Sign up for GitHub

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

Jump to bottom

fix Dependabot alerts #9

Closed
cz4rs opened this issue Dec 7, 2023 · 1 comment · Fixed by #33
Closed

fix Dependabot alerts #9

cz4rs opened this issue Dec 7, 2023 · 1 comment · Fixed by #33
Labels
pri: low Low priority task:3

Comments

@cz4rs
Copy link
Collaborator

cz4rs commented Dec 7, 2023

Dependabot currently shows 13 alerts for the repo (including 6 with high severity), all based on requirements.txt.
Try to fix them by upgrading the dependencies.
@cz4rs cz4rs changed the title fix dependabot alerts fix Dependabot alerts Dec 7, 2023
@cz4rs cz4rs added the pri: low Low priority label Feb 8, 2024
@nmm0 nmm0 added the task:3 label Sep 5, 2024
@nmm0
Copy link
Collaborator

nmm0 commented Sep 5, 2024

We can for now just remove requirements.txt, it's only used for documentation which isn't maintained right now

cz4rs added a commit to NexGenAnalytics/distBVH that referenced this issue Sep 9, 2024
@cz4rs
sandialabs#9: remove requirements.txt
bf1f3f9
@cz4rs cz4rs mentioned this issue Sep 9, 2024
Merged
nmm0 added a commit that referenced this issue Sep 9, 2024
@nmm0
Merge pull request #33 from NexGenAnalytics/9-dependabot-alerts
b660662 
#9: remove `requirements.txt`
@nmm0 nmm0 closed this as completed in #33 Sep 9, 2024
cz4rs added a commit to NexGenAnalytics/distBVH that referenced this issue Sep 9, 2024
@cz4rs
sandialabs#9: remove requirements.txt
3ea3ea7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
pri: low Low priority task:3
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#9: remove requirements.txt NexGenAnalytics/distBVH
2 participants
@cz4rs @nmm0