Removes unused poetry config to unblock Dependabot updates #719
Conversation
Ndiritu
changed the title
|
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
There was a problem hiding this comment.
I don't have any issues with removing poetry from the graph repos.
Simply wanted to point out the configuration is going to be different from the kiota python repo and there are benefits to aligning those things (streamlined tooling, etc...)
Definitely. Creating a new issue to track this. Disabling is a temporary measure to unblock kiota fixes getting to the SDK release & and dependabot upgrades. |
|
Tracking here microsoftgraph/msgraph-sdk-python#969 |
Some required parameters e.g. package name, version, authors for
poetry's configuration are currently missing causing dependabot runs to fail for this repository causing older versions of dependencies to be used by customers.We could add a complete poetry config to fix the issue, but we'd have duplicate config for versions, pkg name etc.
This PR proposes dropping the poetry config since we're not using poetry to manage dependencies, builds/releases.
Poetry required config ref