Skip to content

Sign the workbook : Shambhavi Sharma - #24

Merged
geetxnshgoyal merged 1 commit into
NST-DEVFORGE:mainfrom
ShambhaviSharma28:sign/ShambhaviSharma28
Sep 22, 2026
Merged

geetxnshgoyal merged 1 commit into
NST-DEVFORGE:mainfrom
ShambhaviSharma28:sign/ShambhaviSharma28

Conversation

@ShambhaviSharma28

Copy link
Copy Markdown
Contributor

Signing the workbook

Name: Shambhavi Sharma
Github: ShambhaviSharma28

Checklist

  • [✓] I worked on a branch (sign/<username>), not on main
  • [✓] Files changed shows exactly one file: signatures/<my-username>.md
  • [✓] The - **GitHub:** @username line matches my account
  • [✓] I read the README steps top to bottom

Which step did you have to look up?

I had to look up for what Branches are.

@geetxnshgoyal
geetxnshgoyal requested review from a team, geetxnshgoyal and sharmavikas18 and removed request for a team September 22, 2026 08:48

@geetxnshgoyal geetxnshgoyal left a comment

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.

Good one, Shambhavi. Small polish: the line reads "I'm here to: to immerse…", so drop the extra "to". And the quotes around "nothing yet" aren't needed. Small details, but reading your own diff before you open a PR catches exactly these.

Approved and merged. ✅ Next: open the workbook, submit milestone 1 with this PR's link, and write your reflection within 48 hours.

@geetxnshgoyal
geetxnshgoyal merged commit 51194f4 into NST-DEVFORGE:main Sep 22, 2026
1 check 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