Skip to content

Fixed syntax error in the docstring of expense_tracker.py#686

Open
shivani406 wants to merge 1 commit into
komalharshita:mainfrom
shivani406:fix/expense_tracker.py
Open

Fixed syntax error in the docstring of expense_tracker.py#686
shivani406 wants to merge 1 commit into
komalharshita:mainfrom
shivani406:fix/expense_tracker.py

Conversation

@shivani406
Copy link
Copy Markdown

Summary [required]

On running expense_tracker.py file, It shows and error at Line no 16.
The starting quote of docstring was missing, which was causing the error.
Fixed the docstring so that expense_tracker.py runs without any errors.

Related Issue [required]

Closes issue #670

Type of Change [required]

  • [ ✔] Bug fix — resolves a broken behaviour
  • Feature — adds new functionality
  • Data — adds new projects to data/projects.json
  • Documentation — updates docs, README, or code comments only
  • Style — CSS or visual changes only, no logic change
  • Refactor — restructures code without changing behaviour
  • Test — adds or updates tests

What Was Changed [required]

expense_tracker.py was changed, Added missing starting quote of docstring, Syntax error corrected

File Change made
expense_tracker.py Fixed missing docstring

How to Test This PR [required]

  1. Clone this branch: git checkout your-branch-name
  2. Install dependencies: pip install -r requirements.txt
  3. Run the app: python app.py
  4. Open http://127.0.0.1:5000 and...
  5. Run the tests: python tests/test_basic.py

Test output:
40 passed, 0 failed out of 40 tests

Test Results [required]

expense_tracker.py runs without any error
passed_test_cases

Self-Review Checklist [required]

  • [✔ ] I have read CONTRIBUTING.md and followed all guidelines
  • [ ✔] My branch name follows the convention: feat/, fix/, docs/, data/, style/, test/
  • [ ✔] I have run python tests/test_basic.py and all 27 tests pass
  • [ ✔] I have run flake8 . locally and there are no errors
  • [ ✔] I have not introduced any print() or console.log() debug statements
  • [ ✔] Every new function I wrote has a docstring
  • [ ✔] I have not modified files outside the scope of the linked issue
  • [ ✔] If I changed the UI, I tested it at 375px (mobile) and 1280px (desktop)
  • [ ✔] If I added a project to the dataset, it has all required JSON fields

Notes for Reviewer

None

@vercel
Copy link
Copy Markdown

vercel Bot commented May 29, 2026

@shivani406 is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added gssoc-2026 type:bug Something isn't working labels May 29, 2026
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for submitting your first pull request to DevPath.

Before review:

  • Complete the PR template fully
  • Ensure all tests pass
  • Link your PR to an issue
  • Keep changes scoped to the issue

A maintainer will review your contribution soon.

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

Labels

gssoc-2026 type:bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant