Skip to content

Sign/skmdfirdousf34 - #14

Merged
geetxnshgoyal merged 4 commits into
NST-DEVFORGE:mainfrom
skmdfirdousf34:sign/skmdfirdousf34
Sep 23, 2026
Merged

geetxnshgoyal merged 4 commits into
NST-DEVFORGE:mainfrom
skmdfirdousf34:sign/skmdfirdousf34

Conversation

@skmdfirdousf34

@skmdfirdousf34 skmdfirdousf34 commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

Signing the workbook

Name: Sk Md Firdous
GitHub username: skmdfirdousf34

Checklist

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

Which step did you have to look up?

@geetxnshgoyal

Copy link
Copy Markdown
Member

Hi @skmdfirdousf34 👋 thanks for signing the workbook! The Validate signature check failed. Here's everything to fix in one go:

  • The first line still says # Your Name. Replace Your Name with your real name, for example # Asha Rao (keep the # ).

How to fix it (in your clone, on the same branch as this PR):

# edit signatures/skmdfirdousf34.md and fix the lines above, then:
git add -A
git commit -m "Fix signature"
git push

Your finished file should look like this:

# Your Real Name

- **GitHub:** @skmdfirdousf34
- **Batch:** 2026
- **I'm here to:** your own line
- **One thing I've built:** a link, or nothing yet

Push to this same PR. Don't open a new one. The check re-runs by itself, and a governance reviewer is requested once it's green. Stuck? See Stuck? in the README or ask in the club group.

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

github-actions Bot commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

✅ @skmdfirdousf34, your signature passed the check. The automatic merge didn't go through, so a council member will merge it shortly.

A few polish tips for next time (no need to change this PR):

  • Start links with https:// so they're clickable.

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

Great signature, Firdous, and good that you named GSoC as a goal — the GSoC Playbook is worth a read. Tiny tip: write links with https:// (https://skmdfirdous.vercel.app) so they're clickable.

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

@geetxnshgoyal
geetxnshgoyal merged commit 5bc3b93 into NST-DEVFORGE:main Sep 23, 2026
4 checks passed
geetxnshgoyal added a commit that referenced this pull request Sep 23, 2026
GitHub treats a PR whose branch carries an older copy of a workflow file
as one that modifies workflows, and then refuses to let GITHUB_TOKEN
merge it — stranding every fork made before the change. That bit #17, #14
and #7 today.

Only .github/workflows is restricted, so the logic moves to
.github/scripts/ (validate-signature.sh, workbook-bot.js) and the
workflow becomes a thin runner. Editing the logic from now on leaves
existing forks mergeable.

Also folds in #63: on a 403 the bot explains the stale fork and how to
sync it, instead of a vague hand-off to the council.
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