Skip to content

GitHub authorisations - #450

Merged
Yaswant Pradhan (yaswant) merged 11 commits into
MetOffice:github_wpsfrom
james-bruten-mo:github_auth
Sep 25, 2025
Merged

GitHub authorisations#450
Yaswant Pradhan (yaswant) merged 11 commits into
MetOffice:github_wpsfrom
james-bruten-mo:github_auth

Conversation

@james-bruten-mo

Copy link
Copy Markdown
Contributor

Adding instructions for authenticating with github.
Covering initial setup of git config, ssh keys, verified commits and gh cli.
This is mostly signposting to external pages for setup that will generally be required for simulation systems repos.
Adding a few reviewers to this one to check whether there's anything that's been missed.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'm happy with this. Will leave it unmerged for Yash or Sam to also take a look.

Comment thread source/WorkingPractices/forking.rst Outdated
Comment thread source/WorkingPractices/gh_authorisation.rst
Comment thread source/WorkingPractices/gh_authorisation.rst Outdated
Comment thread source/WorkingPractices/gh_authorisation.rst
Comment thread source/WorkingPractices/gh_authorisation.rst

@yaswant Yaswant Pradhan (yaswant) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good 👍🏽 Few minor comments.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks Yash, all good points and implemented

Comment thread source/WorkingPractices/forking.rst Outdated
Comment thread source/WorkingPractices/gh_authorisation.rst
Comment thread source/WorkingPractices/gh_authorisation.rst Outdated
Comment thread source/WorkingPractices/gh_authorisation.rst
Comment thread source/WorkingPractices/gh_authorisation.rst

@yaswant Yaswant Pradhan (yaswant) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@yaswant
Yaswant Pradhan (yaswant) merged commit ec7281d into MetOffice:github_wps Sep 25, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

crSet This is added to a pull request once the SSD team have assigned a code reviewer.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants