Skip to content

Lab 03 solution - Aleksei Fominykh#59

Closed
ph1larmon1a wants to merge 5 commits intoinno-devops-labs:mainfrom
ph1larmon1a:feature/lab3
Closed

Lab 03 solution - Aleksei Fominykh#59
ph1larmon1a wants to merge 5 commits intoinno-devops-labs:mainfrom
ph1larmon1a:feature/lab3

Conversation

@ph1larmon1a
Copy link

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.

Local PR: ph1larmon1a#4

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.

2 participants