fix: teaching elswhere questions #19
Conversation
📝 WalkthroughWalkthroughThe changes revise the "Teaching_Elsewhere.md" documentation to correct spelling errors, clarify phrasing, add specific details about equipment requirements and power outlets, and refine descriptions of course durations and terminology. No structural changes or modifications to code entities were made. Changes
Sequence Diagram(s)No sequence diagram is generated as the changes are limited to documentation corrections and clarifications. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (7)
Teaching_Elsewhere.md (7)
3-3: Refine “partially” placement for clarity
The clause “which answers these questions, partially” is awkwardly placed—consider:
- “unless a public documentation link is provided that partially answers these questions.”
7-7: Simplify phrasing to active voice
“This questionnaire helps to ask the right questions” reads well, but “helps you ask” is more direct:
- “This questionnaire helps you ask the right questions:”
11-11: Ensure consistent pluralization and natural phrasing
Use plural for all examples and a more natural interrogative:
- “(e.g. for bioinformaticians, physicists, etc.): Does the organization employ a support scientist for this topic? If so, what are their contact details?”
12-12: Shorten and strengthen the instruction
Combine and tighten the wording:
- “Inform all participants that a GitHub account is required.”
21-21: Reword the power outlet question
“Possibly:” is vague—consider merging the lines for flow:
- “When teaching abroad, what is the standard for power outlets?”
36-36: Consistent heading punctuation
Other section headers end with a colon. Add “:” to match style:
- “- Cluster Specialities:”
46-46: Refine parenthetical note
Capitalize and separate the parenthetical or move it into the body:
- “How long will the course run (number of days)? Note: Our minimum is 2 days for the full creator course and 1 day for the user-only course.”
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
Teaching_Elsewhere.md(3 hunks)
🧰 Additional context used
🪛 LanguageTool
Teaching_Elsewhere.md
[style] ~17-~17: Consider using a more polite way of asking this question.
Context: ...ble? - Is there a "Presenter PC"? - Is it possible to plug in a laptop? (HDMI and/or VGA?)...
(IS_IT_POSSIBLE_TO)
[uncategorized] ~20-~20: Possible missing comma found.
Context: ... paired programming possible during the course considering the classroom's seating? ...
(AI_HYDRA_LEO_MISSING_COMMA)
⏰ Context from checks skipped due to timeout of 90000ms (2)
- GitHub Check: build
- GitHub Check: build
🔇 Additional comments (1)
Teaching_Elsewhere.md (1)
19-19: Approve adaptor clarification
The change to specify “video projector” is clear and consistent.
Artifacts |
|
@FlorianBoecker the remark about the github account is related to an upcoming change |
various little fixes for the question set when teaching elsewhere
Summary by CodeRabbit