Skip to content

Conversation

james-bruten-mo
Copy link
Contributor

An initial attempt at github working practices. This is already getting a bit large so probably time to bank some changes.

The structure is:

  • modify Working Practices lander page
  • Unwritten page on authorising for github and ssh keys
  • New page on forks
  • New page on beginning a gh development, up to point of linking to the development section.
  • Not done - change reviews.rst to include opening a PR, something on CI, any other changes from current

(I've also included my flow chart, as much because I had it to hand as anything. It's in the location where I think it wants to live, but obviously will probably change as we improve it).

@yaswant
Copy link
Contributor

yaswant commented Jul 14, 2025

@james-bruten-mo do you have a preview page to look at internally?

@yaswant yaswant added this to the Git Migration milestone Jul 14, 2025
@james-bruten-mo
Copy link
Contributor Author

@james-bruten-mo do you have a preview page to look at internally?

Sure, see https://wwwspice/~james.bruten/gh_wps/index.html

Copy link
Collaborator

@jennyhickson jennyhickson left a comment

Choose a reason for hiding this comment

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

Some minor comments, but a great start getting the bones and structure in and we can iterate on it from here.

james-bruten-mo and others added 8 commits July 14, 2025 09:50
Co-authored-by: Jenny Hickson <61183013+jennyhickson@users.noreply.github.com>
Co-authored-by: Jenny Hickson <61183013+jennyhickson@users.noreply.github.com>
Co-authored-by: Jenny Hickson <61183013+jennyhickson@users.noreply.github.com>
Co-authored-by: Jenny Hickson <61183013+jennyhickson@users.noreply.github.com>
Co-authored-by: Jenny Hickson <61183013+jennyhickson@users.noreply.github.com>
Co-authored-by: Jenny Hickson <61183013+jennyhickson@users.noreply.github.com>
Co-authored-by: Jenny Hickson <61183013+jennyhickson@users.noreply.github.com>
Copy link
Collaborator

@jennyhickson jennyhickson left a comment

Choose a reason for hiding this comment

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

Thanks.

@jennyhickson jennyhickson merged commit 45db38d into MetOffice:github_wps Jul 14, 2025
2 checks passed
@james-bruten-mo james-bruten-mo deleted the initial_gh_wps branch July 14, 2025 10:26
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.

3 participants