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

Apcsp framework updates #399

Merged
merged 3 commits into from Dec 12, 2016

Conversation

2 participants
@ptymann
Contributor

ptymann commented Oct 30, 2016

Proposed changes

Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request:

Updating this guide to match the minor changes to the curriculum in the Fall 2016 version.

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the pull request. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your change.

  • I have read the contribution guidelines
  • I have linked any relevant existing issues/suggestions in the description above (include #??? in your description to reference an issue, where ??? is the issue number)
  • I have applied the relevant labels for this change
  • I have run the generation script and checked the output
  • I have added necessary documentation (if appropriate)

Further comments

If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc. Feel free to add any images that might be helpful to understand the initial problem/solution.

ptymann added some commits Oct 24, 2016

text/en/curriculum-guides/apcsp/guide-programming-introduction.md
Updated to reflect changes made to the final AP-CS curriculum framework
Removed the text 'We are currently updating these guides to match the…
… minor changes to the curriculum

in the Fall 2016 version.' since we made the updates.

`cs-field-guide/text/en/curriculum-guides/apcsp/index.md`
@JackMorganNZ

This comment has been minimized.

Show comment
Hide comment
@JackMorganNZ

JackMorganNZ Nov 6, 2016

Member

I'll have a proper look at this later this week, but looks great from a quick glance.

Member

JackMorganNZ commented Nov 6, 2016

I'll have a proper look at this later this week, but looks great from a quick glance.

@JackMorganNZ JackMorganNZ merged commit b163938 into uccser:develop Dec 12, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@JackMorganNZ JackMorganNZ added this to the v2.9.0 - Pōhutukawa milestone Dec 12, 2016

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