Skip to content

docs : fix resources#8

Open
Kalyani-Puranik wants to merge 5 commits intomainfrom
fix-resources
Open

docs : fix resources#8
Kalyani-Puranik wants to merge 5 commits intomainfrom
fix-resources

Conversation

@Kalyani-Puranik
Copy link
Copy Markdown
Contributor


Summary

Added weekly report for Wek 2 and made requested changes in my Week0.md

Brief description of what this PR does.


Type

  • feat: new feature
  • fix: bug fix
  • docs: documentation
  • refactor: code restructuring
  • test: tests only
  • chore: maintenance

Changes

  • Added space after 2 hashes in Week0.md
  • Added my resources and daily report for week 2

Testing

Describe how you tested this:

  • Unit tests added/updated
  • Manual testing performed
  • All tests pass locally

Checklist

  • Code follows project style
  • Commit messages follow COMMIT_CONVENTION.md
  • No API keys or sensitive data in code

@Kalyani-Puranik Kalyani-Puranik changed the title Fix resources docs : resources fixed Mar 3, 2026
@Kalyani-Puranik Kalyani-Puranik changed the title docs : resources fixed docs : fix resources Mar 3, 2026
@Kalyani-Puranik
Copy link
Copy Markdown
Contributor Author

@mebinthattil

Copy link
Copy Markdown
Member

@mebinthattil mebinthattil left a comment

Choose a reason for hiding this comment

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

@Kalyani-Puranik please check the comment and make the change. Request a re-review once you've pushed the changes.

@@ -0,0 +1,6 @@
# Week 0
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is Week3.md why does the heading say Week 0, please fix that. And please follow the same format of mentioning each day's work for Week3 as well.

Copy link
Copy Markdown
Member

@mebinthattil mebinthattil left a comment

Choose a reason for hiding this comment

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

I'll merge this once Week3 is complete, else remove Week3 and add that using another PR later

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