Skip to content

Fix payload contribution Git instructions - #66

Open
Ashishchoudhary07 wants to merge 1 commit into
OWASP:websitefrom
Ashishchoudhary07:fix/contributing-fork-workflow
Open

Ashishchoudhary07 wants to merge 1 commit into
OWASP:websitefrom
Ashishchoudhary07:fix/contributing-fork-workflow

Conversation

@Ashishchoudhary07

Copy link
Copy Markdown

Summary

  • Fix the payload contribution guide to use the contributor's fork instead of pushing directly to the upstream repository.
  • Add instructions to configure the upstream remote and fetch the latest payload collection.
  • Update branch creation and push commands to follow the correct fork-based workflow.
  • Clarify that pull requests should target the upstream repository's main branch.

Testing

  • git diff --check — passed
  • npx tsc --noEmit — passed
  • npm run lint — passed
  • npx next build --webpack — passed
  • py -3 scripts/validate_website_export.py — passed (47 HTML pages and 2,955 local references validated)

AI Disclosure

  • AI assistance: Yes
  • Tool or model: ChatGPT
  • Provider: OpenAI
  • Contribution: Assisted with drafting and reviewing the Git workflow documentation.
  • Human validation: Reviewed the changes and ran the listed validation commands.

Fixes #63

@github-actions

Copy link
Copy Markdown
Contributor

✅ No duplicate payload entries detected

Checked 0 normalized entries from added/modified payload files on the latest push.

Posted automatically by the Duplicate Payload Check workflow.

This branch has not been deployed

No deployments
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.

1 participant