-
Notifications
You must be signed in to change notification settings - Fork 25
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
use only submodules for noresm checkout #407
use only submodules for noresm checkout #407
Conversation
…preparation for git fleximod
…preparation for git fleximod
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of years ago it seemed GitHub was recommending to go away from https and using tokens or ssh, if I remember correctly. If https still works for all practical purposes, I have no objections.
You can not push changes to github over https (or maybe it requires 2FA identification?). As long as you only check out the code it should be fine. For code developers it is probably easier to use ssh or tokens (although, I haven't really looked into the option of using tokens). |
@TomasTorsvik, @matsbn - thank you for the quick response to this. |
You can push changes to GitHub over HTTPS. Quoting from the NorESM2.1 documentation
Do we need to enhance this section? |
My mistake. I just never took the time to look into the tokens option properly. The same for the github-cli option. Locally, I'm only using SSH. I have the impression that github changes the access options from time to time, so I think it's better to point to github documentation in this case. |
@gold2718 that means to copy github-credentials (PAT) to the HPC system, right (or is there a different way that I am unaware of)? Thus far, I also use ssh agent to access the repos. Maybe worth an edit into the docu (after testing), it seems to be easy to exchange default
(untested thus far, though - just found it, when the question came up on my side). |
This PR does several things.