Skip to content

Fix longest unique run - #111

Merged
pushpam2404 merged 1 commit into
github-community-gitam:mainfrom
debatranayak111:fix/issue-88
Oct 5, 2026
Merged

pushpam2404 merged 1 commit into
github-community-gitam:mainfrom
debatranayak111:fix/issue-88

Conversation

@debatranayak111

Copy link
Copy Markdown
Contributor

Which issue does this fix?

Closes 88

Which file did you change?

What was wrong, and what did you change?

current = current[current.index(value) :] to current = current[current.index(value) + 1 :]
if len(current) < len(best): to if len(current) > len(best):

Terminal output

python issue-88.py
All checks passed!

$ python3 python-open-source-challenge/issue-88.py

Checklist

  • I commented /claim on the issue and it is assigned to me
  • My branch is named fix/issue-__ (not main)
  • I changed only the one file this issue is about
  • Running the file prints All checks passed!
  • I did not rewrite the program from scratch
  • I did not edit check_solution()
  • If I used an AI tool, I understand the change and can explain it in review

First time here? Read CONTRIBUTING.md. Stuck? Comment on your issue saying what you have tried — asking is not failing.

@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.

@debatranayak111

Copy link
Copy Markdown
Contributor Author

excited

@pushpam2404
pushpam2404 merged commit b8fd214 into github-community-gitam:main Oct 5, 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.

2 participants