Skip to content

Fix: completed intermediate exercises#96

Merged
AvanishSalunke merged 1 commit into
ProjectX-VJTI:mainfrom
jiyac-code:main
Mar 15, 2026
Merged

Fix: completed intermediate exercises#96
AvanishSalunke merged 1 commit into
ProjectX-VJTI:mainfrom
jiyac-code:main

Conversation

@jiyac-code

Copy link
Copy Markdown
Contributor

📝 CoC Python Workshop – Submission PR


👤 Student Details

  • Name: Jiya Choksi
  • GitHub Username: jiyac-code
  • University ID: 251081017
  • Branch / Section: IT

📂 Submission Details

  • Folder Name: jiyac-code_solutions
  • Number of Files Changed/Added: 13

📄 List of Files Changed/Added

  • jiyac-code_solutions/test_playground/intermediate

🛠 Description of Changes

I have completed all of the intermediate exercises.


✅ Self-Declaration

I confirm that:

  • I have followed the repository submission guidelines.
  • I worked on my own fork.
  • I confirm my folder is correctly named
  • All changes are inside my jiyac-code_solutions folder.
  • I have tested my code before submission.
  • This is my own work to the best of my knowledge.

🎯 Thank you for your submission!

@AvanishSalunke

Copy link
Copy Markdown
Collaborator

Great work!

@AvanishSalunke AvanishSalunke merged commit 34f298c into ProjectX-VJTI:main Mar 15, 2026
1 check passed
@Once-1296

Copy link
Copy Markdown
Collaborator

Within txt handling, lines[line_no] is correct

But ur condition is

if line_no < 0 or line_no > len(lines)

Shouldn't len(lines) be rejected too as it's invalid index?

Also im not sure of it but i think your fibonacci code may bring. However there are different interpretations of the sequence as I have seen different solutions so that one is alright.

In your next PR when submit the next set of exercises (either advanced or miscellaneous), just make the change in indexing in intermediate/txt_handling.py

Good progress btw😁😎

@jiyac-code

Copy link
Copy Markdown
Contributor Author

Ohh, I must have missed that. Will do the required changes👍

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.

3 participants