Skip to content

Conversation

@sanjeevz3009
Copy link
Collaborator

@sanjeevz3009 sanjeevz3009 commented Dec 9, 2025

What is the context of this PR?

Addresses the issue raised by @delterr, #69.

Currently in the project template README.md we have the instruction below, which unfortunately doesn't do anything when followed. This PR fixes this issue by adding a run target in the project template Makefile and to reflect the change adding a main python script which utilises calculator.py.

3. Run the application
make run   

How to review

  • Pull down the branch/ latest changes
  • Generate a new python template repo -> make run -> the script should work as expected (you will see the output from the python script that's being executed)
  • Make sure to test with Poetry and Pipenv

Follow-up Actions

List any follow-up actions (if applicable), like needed documentation updates or additional testing.

@sanjeevz3009 sanjeevz3009 added the enhancement New feature or request label Dec 9, 2025
@sanjeevz3009 sanjeevz3009 added the bug Something isn't working label Dec 9, 2025
MebinAbraham
MebinAbraham previously approved these changes Jan 8, 2026
MebinAbraham
MebinAbraham previously approved these changes Jan 8, 2026
@sanjeevz3009 sanjeevz3009 merged commit e6e060a into main Jan 8, 2026
5 checks passed
@sanjeevz3009 sanjeevz3009 deleted the fix/enable-make-run-within-project-template branch January 8, 2026 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants