Skip to content
This repository was archived by the owner on Apr 28, 2025. It is now read-only.

Conversation

@martialblog
Copy link
Member

This PR refactores the End of the Training.

After the CI Topic there's now a "What have we learned" Section. These slides offer a chance to look back and internalize all the topics covered.

After the review, the trainer can decide how to move forward, with the "further features in git and gitlab" sections.

See #177

@martialblog martialblog self-assigned this Dec 1, 2022
@martialblog martialblog force-pushed the refactor/end-of-training branch from 8ac08be to 79d4dbf Compare December 1, 2022 13:49
@martialblog martialblog requested a review from theFeu December 1, 2022 13:51
@martialblog
Copy link
Member Author

@htriem @mocdaniel Feedback is welcome

Copy link
Collaborator

@theFeu theFeu left a comment

Choose a reason for hiding this comment

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

I think day2/03_CI is not the right place for the review, maybe we can push those into a day2/04_Summary/What_have_we_learned.md and have a 05_Outlook instead?

@martialblog
Copy link
Member Author

Yes absolutely. It was just faster for the draft...

 - Add 'What have we learned?' section after CI
 - Move following slides into an optional "outlook" section
@martialblog martialblog force-pushed the refactor/end-of-training branch from 79d4dbf to a2f4e2c Compare December 1, 2022 19:45
@martialblog
Copy link
Member Author

I restructured the directories

@mocdaniel
Copy link
Member

I like the changes, and with the 'chapter' logic adjusted I don't see anything left to do before merging, unless you got some more changes locally.

@martialblog martialblog marked this pull request as ready for review December 2, 2022 10:25
@martialblog
Copy link
Member Author

Ready to Ship :shipit:

@martialblog
Copy link
Member Author

@htriem What do you think?

@martialblog martialblog merged commit 72c8418 into master Dec 16, 2022
@martialblog martialblog deleted the refactor/end-of-training branch December 16, 2022 13:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants