Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated techical documentation #328

Merged
merged 5 commits into from
Oct 31, 2023

Conversation

keithkennedyHO
Copy link
Contributor

  • remove references to GigHub pages
  • mention codeowners in PR section of CONTRIBUTING.md
  • updated documentation on raising a security issue
  • provide detail of technical decisions related to migrating repo and hosting

Is this pull request a content or a code change? (Please fill in the relevant section and delete the other)

Content change

I can confirm:

  • Content is suitable to open source, i.e.:
    • Content does not relate to unreleased gov policy
    • Content does not refer to anti-fraud mechanisms
    • Content does not include sensitive business logic

- remove references to GigHub pages
- mention codeowners in PR section of CONTRIBUTING.md
- updated documentation on raising a security issue
- provide detail of technical decisions related to migrating repo and hosting
@keithkennedyHO keithkennedyHO linked an issue Oct 30, 2023 that may be closed by this pull request
1 task
@keithkennedyHO keithkennedyHO linked an issue Oct 30, 2023 that may be closed by this pull request
1 task
@edhamiltonHO edhamiltonHO marked this pull request as ready for review October 30, 2023 13:58
@edhamiltonHO edhamiltonHO requested a review from a team as a code owner October 30, 2023 13:58
Copy link
Contributor

@edhamiltonHO edhamiltonHO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@edhamiltonHO edhamiltonHO requested a review from a team October 30, 2023 13:59
@edhamiltonHO edhamiltonHO requested review from robertdeniszczyc2 and a team October 30, 2023 15:14
@keithkennedyHO keithkennedyHO merged commit 2a60e67 into main Oct 31, 2023
2 checks passed
@keithkennedyHO keithkennedyHO deleted the 274-review-technical-documentation-files branch October 31, 2023 08:15
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.

Review technical documentation files
4 participants