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

99 tidy up git learning resources page #126

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

Lsnaathorst1
Copy link
Contributor

@Lsnaathorst1 Lsnaathorst1 commented Dec 2, 2024

Overview of changes

Adding the link to DevOps guidance right at the top of the page in a call-out box. Suggested tidying up the page a little by adding an extra header and moving the email links from the top section to be with the SDT email address below.

Why are these changes being made?

So users can navigate the page a little easier and know what to find in each section, plus make it much easier for users to find the DevOps guidance

Detailed description of changes

Added the DevOps link in a callout at the top of the page. This was originally done for us by an external user as per this PR:
#102. We were unable to approve this as it was forked.

I have then also attempted to tidy up the page a little by adding an extra heading and moving the email address support to be together. I have then removed the bullets from the SDT email and added to the paragraph as they don't format nicely with the other emails there (looks odd that only that one has bullets and is hard to make it clear they are only applying to the SDT email)

Issue ticket number/s and link

#99

Checklist before requesting a review

  • I have checked the contributing guidelines
  • I have checked for and linked any relevant issues that this may resolve
  • I have checked that these changes build locally
  • I understand that if merged into main, these changes will be publicly available

@Lsnaathorst1 Lsnaathorst1 linked an issue Dec 2, 2024 that may be closed by this pull request
@Lsnaathorst1 Lsnaathorst1 marked this pull request as ready for review December 4, 2024 14:24
@jen-machin jen-machin self-requested a review December 11, 2024 10:37
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.

Tidy up Git learning resources page
2 participants