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

Courses: Remove from repo and specify commit to view them #1020

Merged
merged 3 commits into from Jun 27, 2018

Conversation

Projects
None yet
2 participants
@bradjc
Copy link
Contributor

bradjc commented Jun 22, 2018

Pull Request Overview

The SOSP course hasn't compiled in months, the Sensys course doesn't work with the c userland being moved, and maintaining the courses is a difficult task. From #987, it sounds like our best bet is to point users to a working commit rather than try to keep the courses up to date, particularly since we are going to develop more this year.

This removes the courses and updates the readmes to point to a commit to checkout where the tutorial works (compiles).

It doesn't touch the rustconf one because that one doesn't have any code in the repo.

Testing Strategy

n/a

TODO or Help Wanted

n/a

Documentation Updated

  • Updated the relevant files in /docs, or no updates are required.

Formatting

  • Ran make formatall.

@bradjc bradjc referenced this pull request Jun 22, 2018

Closed

Tracking: Complete move of userland to libtock-c #970

5 of 5 tasks complete

@ppannuto ppannuto added the P-Upkeep label Jun 26, 2018

@bradjc bradjc force-pushed the sosp-course-commit branch 2 times, most recently from ec3a454 to e98f21d Jun 26, 2018

bradjc added some commits Jun 22, 2018

doc: courses: specify working commit
This removes the SOSP course and instead points users to a commit hash
they can checkout where the tutorial actually builds.
doc: courses: specify commit for sensys course
Point users to where they can go to view the working sensys course.

@bradjc bradjc force-pushed the sosp-course-commit branch from e98f21d to 7796da2 Jun 27, 2018

@ppannuto
Copy link
Member

ppannuto left a comment

Just a rebase from last approval.

@ppannuto ppannuto merged commit e86e6b1 into master Jun 27, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
deploy/netlify Deploy preview ready!
Details

@ppannuto ppannuto deleted the sosp-course-commit branch Jun 27, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment