-
Notifications
You must be signed in to change notification settings - Fork 924
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
Update FAQ about 22.01 to 23.0 #5285
Conversation
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.
Thanks @lldelisle! LGTM, just one small suggestion.
Co-authored-by: Björn Grüning <[email protected]>
2. git checkout step-4 | ||
3. Diff and sync (e.g. `vimdiff group_vars/galaxyservers.yml git-gat/group_vars/galaxyservers.yml`) for the main configuration files: | ||
2. cd git-gat | ||
3. git checkout step-4 |
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.
we might want to replace this with a commit, because the step-4 tag will evolve over time. might want to go back in time to 2023's version of this file to link to, hmm.
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.
OK I will find it.
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.
This is the step 4 of the first time git-gat has the release 23.0: a3324eef24445646325790489b8f572d9a7581c1 849c30058c5010440f2a0787ea053bfea530111a
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.
(ah those are trees not commit IDs), yeah that's the start and I guess we made a lot of changes over prep for that course, how about git checkout c2e7bf6d3584fbf3281fb57d8024a9189f957e0e
which should be at the end of that series of updates and hopefully captures anything we discovered during testing.
Co-authored-by: Lucille Delisle <[email protected]>
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.
Thank you so much for adding all of this extra detail!
Thanks to you for providing this FAQ. This was really helpful to do it. |
No description provided.