-
Notifications
You must be signed in to change notification settings - Fork 50
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
Add cheatsheet and more videos to README #837
Conversation
@Melkiades which 2 videos are you referencing here (#834 (comment))? |
Code Coverage Summary
Diff against main
Results for commit: 0cd2564 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Unit Tests Summary 1 files 24 suites 1m 39s ⏱️ Results for commit 0cd2564. ♻️ This comment has been updated with latest results. |
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.
I thought about it and I think we may want to add it to the top bar or do you think it is too crowded?
Regarding the second half of the issue, there are a couple of videos from Gabe and me that are not present in the readme and I think it could be useful to add them. When you are online I can add them or send them to you ;)
https://www.youtube.com/watch?v=sxFsavKI7s4
https://www.youtube.com/watch?v=zBm_NZ0VtKs (this to add to the slides)
https://www.youtube.com/watch?v=1i6vOId2h4A (this is new to me)
https://www.youtube.com/watch?v=YPmbLPSYFYM (our demo that should go here but also on tern/TLG-c/falcon maybe? I would add the slides too)
https://www.youtube.com/watch?v=yZS4OBuJe_Q (this is the UseR! conference that should be very visible imo bc there are also other people and the discussion seemed very interesting to me)
Maybe @telepath37 you know better where to place the videos and slides. Should they be on the documentation? on the website?
ps: this should also be more present in general: https://rconsortium.github.io/rtrs-wg/tablepkgs.html#rtables
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.
lgtm!
Unit Test Performance Difference
Results for commit 635054e ♻️ This comment has been updated with latest results. |
Closes #834