Skip to content

Fix issue 67 - #117

Merged
pushpam2404 merged 1 commit into
github-community-gitam:mainfrom
prmanna78-source:fix/issue-67
Oct 8, 2026
Merged

pushpam2404 merged 1 commit into
github-community-gitam:mainfrom
prmanna78-source:fix/issue-67

Conversation

@prmanna78-source

Copy link
Copy Markdown
Contributor

Which issue does this fix?

Closes #67

Which file did you change?

python-open-source-challenge/issue-67.py

What was wrong, and what did you change?

The program was checking whether the marks decreased instead of increased in every successive test. I changed the comparison so that each mark must be greater than the previous mark. I also added a check to handle students with fewer than two tests.

Terminal output

$ python3 python-open-source-challenge/issue-67.py
All checks passed!

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Congratulations on your first pull request to this repository.

Here is what happens next:

  1. The check below runs only the file you changed and needs it to print All checks passed!. If it is red, click Details — it shows the same error you would see by running the file yourself
  2. A maintainer reviews within 48 hours and may leave line comments
  3. Once it is approved, a maintainer merges it

Two things that get PRs closed, so please double-check:

  • No Closes #<issue> in the description
  • The issue was never assigned to you

Nice work getting this far. Most people never open their first PR.

@pushpam2404
pushpam2404 merged commit 553b24d into github-community-gitam:main Oct 8, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Issue 68: Report total sales, average sales and the top performer

2 participants