-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Closed
aws/aws-cdk-cli
#1047Labels
effort/smallSmall work item – less than a day of effortSmall work item – less than a day of effortfeature-requestA feature should be added or improved.A feature should be added or improved.good first issueRelated to contributions. See CONTRIBUTING.mdRelated to contributions. See CONTRIBUTING.mdp2package/toolsRelated to AWS CDK Tools or CLIRelated to AWS CDK Tools or CLI
Description
cdk init --language python generates setup.py file, and requirements.txt file with -e ., that installs the project in editable mode (i.e. setuptools “develop mode”) from local path. This approach is used for distributing Python code to developers - a reusable construct in CDK case. CDK application is not a reusable construct, hence shouldn't have setup.py, and only list its direct dependencies in requirements.txt.
Use Case
Make cdk init --language python scaffolding more aligned with Python guidance on packaging applications as compared to packaging libraries and tools
Proposed Solution
Remove setup.py and -e . from requirements.txt when running cdk init --language python
Other
- 👋 I may be able to implement this feature request
-
⚠️ This feature might incur a breaking change
This is a 🚀 Feature Request
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
effort/smallSmall work item – less than a day of effortSmall work item – less than a day of effortfeature-requestA feature should be added or improved.A feature should be added or improved.good first issueRelated to contributions. See CONTRIBUTING.mdRelated to contributions. See CONTRIBUTING.mdp2package/toolsRelated to AWS CDK Tools or CLIRelated to AWS CDK Tools or CLI