Skip to content
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

docs(sessions): 📝 include learning outcomes for course #46

Merged
merged 3 commits into from
Oct 7, 2024

Conversation

lwjohnst86
Copy link
Member

Description

Closes #4

Reviewer Focus

This PR needs an in-depth review.

Checklist

  • Ran spell-check
  • Formatted Markdown
  • Rendered website locally

Copy link
Member

@signekb signekb left a comment

Choose a reason for hiding this comment

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

Very nice 🌟 I have some suggestions, mostly minor language-related things.

preamble/syllabus.qmd Outdated Show resolved Hide resolved
Comment on lines 18 to 21
organizationally in order to work well together as a team. (*this is
to remove thoughts people have that "this doesn't work for me, I
work best in chaos", to emphasize, it isn't about individuals, it's
about the group*)
Copy link
Member

Choose a reason for hiding this comment

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

Should this parenthesis be removed? 😅

Suggested change
organizationally in order to work well together as a team. (*this is
to remove thoughts people have that "this doesn't work for me, I
work best in chaos", to emphasize, it isn't about individuals, it's
about the group*)
organizationally in order to work well together as a team.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, it was more for you and me to be clear about, I guess I should have written that as a PR comment 😛 ooppsss

preamble/syllabus.qmd Outdated Show resolved Hide resolved
preamble/syllabus.qmd Outdated Show resolved Hide resolved
preamble/syllabus.qmd Outdated Show resolved Hide resolved
preamble/syllabus.qmd Outdated Show resolved Hide resolved
preamble/syllabus.qmd Show resolved Hide resolved
preamble/syllabus.qmd Outdated Show resolved Hide resolved
preamble/syllabus.qmd Outdated Show resolved Hide resolved
Comment on lines 18 to 21
organizationally in order to work well together as a team. (*this is
to remove thoughts people have that "this doesn't work for me, I
work best in chaos", to emphasize, it isn't about individuals, it's
about the group*)
Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, it was more for you and me to be clear about, I guess I should have written that as a PR comment 😛 ooppsss

preamble/syllabus.qmd Show resolved Hide resolved
preamble/syllabus.qmd Outdated Show resolved Hide resolved
preamble/syllabus.qmd Outdated Show resolved Hide resolved
@lwjohnst86 lwjohnst86 merged commit 6252366 into main Oct 7, 2024
@lwjohnst86 lwjohnst86 deleted the docs/add-learning-outcome branch October 7, 2024 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Brainstorm and create draft of the course learning outcome and objectives
2 participants