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

Update lab b9 #216

Merged
merged 1 commit into from
Aug 28, 2020
Merged

Update lab b9 #216

merged 1 commit into from
Aug 28, 2020

Conversation

edricx
Copy link
Contributor

@edricx edricx commented Jul 30, 2020

  • Rewrote the Git exercise to be more detailed and use a Python example program instead of C.

  • Added an exercise to make a dummy PR on Github. The dummy repo still needs to be set up and the lab instructions updated with the repo URL for this semester.

Related PR to update the lab b9 example code: 0xcf/decal-labs#21

@edricx edricx linked an issue Jul 30, 2020 that may be closed by this pull request
@BernardZhao
Copy link
Member

Looks good so far, your formatting is excellent. What repo did you think about having dummy PRs on?

@edricx
Copy link
Contributor Author

edricx commented Aug 6, 2020

I was thinking we just set up a repo called b9-fall2020 or something that has a simple text file. We can have them fork it, write their name in the file, and submit a PR. Point out different features along the way like linking issues, making comments, rich diff, etc. We can get a headcount of who finished the task by the number of open PRs.

@BernardZhao
Copy link
Member

Sounds good, feel free to go ahead with this asap.

Copy link
Member

@64bitpandas 64bitpandas left a comment

Choose a reason for hiding this comment

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

Looks awesome!

Some overall comments (also check individual comments):

  • We should sprinkle in simple Gradescope progress check questions so students have something more concrete to submit.
  • Make sure to make the dummy repo and fill in all the info before merging!

lab will be in the `b9` directory.

### Part 1: Git Basics

Copy link
Member

Choose a reason for hiding this comment

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

Insert a super short blurb reminding people of what git is, maybe link to an intro resource or lecture

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll probably make another PR to add the lecture link once it's been uploaded.

labs/b9.md Show resolved Hide resolved
labs/b9.md Outdated Show resolved Hide resolved
labs/b9.md Show resolved Hide resolved
labs/b9.md Show resolved Hide resolved
labs/b9.md Show resolved Hide resolved
- Rewrite core Git exercise with Python example

- Add optional interactive rebase section

- Add Gradescope checks

- Add exercise to create a dummy pull request
@edricx edricx merged commit c53e90f into 0xcf:master Aug 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Update b9
3 participants