Skip to content

Make use of poetry #122

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
Open
wants to merge 31 commits into
base: main
Choose a base branch
from
Open

Make use of poetry #122

wants to merge 31 commits into from

Conversation

nezgrath
Copy link
Collaborator

Pull Request Template

Description

Please provide a brief description of the changes in this pull request.

Checklist

Please complete the following checklist before submitting your pull request:

  • I have tested these changes locally and verified that they work as intended.
  • I have updated any documentation as needed to reflect these changes (if appropriate)
  • I have verified that these changes to the best of my knowledge do not introduce any security vulnerabilities.
  • Unit tests have been added (if appropriate)
  • Test-cases have been added (if appropriate)
  • Testing instructions have been added (if appropriate)

@github-actions github-actions bot added dependencies Related with project dependencies maintenance Package and maintenance related labels Nov 28, 2024
@wiz-inc-2dcf8239e9
Copy link

wiz-inc-2dcf8239e9 bot commented Jan 22, 2025 •
edited
Loading

Wiz Scan Summary

Scanner Findings
Vulnerability Finding Vulnerabilities 1 Low
Data Finding Sensitive Data
Secret Finding Secrets
IaC Misconfiguration IaC Misconfigurations
Total 1 Low

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension.

@nezgrath nezgrath marked this pull request as ready for review January 30, 2025 04:49
@FedericoNegri
Copy link
Collaborator

@nezgrath why adding poetry to this repo now when Manikanth is anyway reworking its structure for open sourcing?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ojkoenig ojkoenig Awaiting requested review from ojkoenig ojkoenig is a code owner

@FedericoNegri FedericoNegri Awaiting requested review from FedericoNegri FedericoNegri is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
dependencies Related with project dependencies maintenance Package and maintenance related
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nezgrath @FedericoNegri