Skip to content

Feature/lab3#4

Open
ph1larmon1a wants to merge 3 commits intomainfrom
feature/lab3
Open

Feature/lab3#4
ph1larmon1a wants to merge 3 commits intomainfrom
feature/lab3

Conversation

@ph1larmon1a
Copy link
Owner

Goal

Add lab 3 submission materials and demonstrate use of the standardized PR template & checklist.

Changes

  • Added labs/submission3.md documenting pre-commit secret scanning setup, tests, and analysis.
  • Added .github/pull_request_template.md for consistent PR workflow.
  • Created feature/lab3 branch and opened PR into main to validate auto-fill.

Testing

  • Verified that commits are signed with SSH and display the Verified badge on GitHub.
  • Verified .git/hooks/pre-commit accepts commits containing fake AWS credentials(maybe need real ones).
  • Confirmed successful commit after secrets were removed.
  • Opened this PR and confirmed template auto-filled with Goal/Changes/Testing sections and checklist.
  • Ticked all checklist items to show compliance.

Checklist

  • PR has a clear and descriptive title
  • Documentation updated if needed
  • No secrets or large temporary files committed
  • Task 1 done
  • Task 2 done
  • Task 3 done

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