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

Principles Documentation #375

Merged
merged 21 commits into from
Apr 12, 2020
Merged

Principles Documentation #375

merged 21 commits into from
Apr 12, 2020

Conversation

pathunstrom
Copy link
Collaborator

@pathunstrom pathunstrom commented Oct 29, 2019

This is a rough first draft, would love some input on any of these points, or if anyone thinks I'm missing something key.

  • Students and Learners First
  • Creativity without Limits
  • Fun
  • Embracing Radical Change

@pathunstrom pathunstrom added the discussion Issues that more just discussing than specific flaws or features label Oct 29, 2019
@pathunstrom pathunstrom requested a review from AstraLuma October 29, 2019 16:22
@pathunstrom pathunstrom self-assigned this Oct 29, 2019
docs/discussion/values/community-values.rst Outdated Show resolved Hide resolved
docs/discussion/values/technical-values.rst Outdated Show resolved Hide resolved
docs/discussion/values/technical-values.rst Outdated Show resolved Hide resolved
docs/discussion/values/technical-values.rst Outdated Show resolved Hide resolved
docs/discussion/values/technical-values.rst Outdated Show resolved Hide resolved
docs/discussion/values/technical-values.rst Outdated Show resolved Hide resolved
Copy link
Member

@AstraLuma AstraLuma left a comment

Choose a reason for hiding this comment

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

Some copy editing

docs/discussion/values/index.rst Outdated Show resolved Hide resolved
docs/discussion/values/index.rst Outdated Show resolved Hide resolved
docs/discussion/values/index.rst Outdated Show resolved Hide resolved
docs/discussion/values/index.rst Outdated Show resolved Hide resolved
@pathunstrom
Copy link
Collaborator Author

Leaving a note here: I'm changing Accepting Radical Change to Radical Acceptance, and containing Accepting Change under it.


Our focus on creativity without limits is about supporting users at all
skill levels, and to help guide them from their first lines of code through
contributing to open source.
Copy link
Member

Choose a reason for hiding this comment

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

Is this really the end goal?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It is explicitly part of the pitch I've been using in discussions with people.

Support All Users
----------------------

We're Students First, but students aren't students forever, and we want
Copy link
Member

Choose a reason for hiding this comment

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

A+

develop ``ppb`` users.

From the user perspective, clear tutorials and example code. Open
discussions about the design of the system. Type hinting to allow the tools to
Copy link
Member

Choose a reason for hiding this comment

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

Ehhh...... type hinting has proven to be extremely difficult for us.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Still a thing we put effort into.

@pathunstrom pathunstrom marked this pull request as ready for review April 12, 2020 18:55
@pathunstrom pathunstrom requested a review from a team as a code owner April 12, 2020 18:55
@AstraLuma
Copy link
Member

bors r+

@pathunstrom pathunstrom added bors Someone has bors r+ this PR and removed discussion Issues that more just discussing than specific flaws or features labels Apr 12, 2020
@bors
Copy link
Contributor

bors bot commented Apr 12, 2020

@bors bors bot merged commit c168f02 into ppb:master Apr 12, 2020
@pathunstrom pathunstrom deleted the principles-docs branch April 15, 2020 00:44
@AstraLuma AstraLuma mentioned this pull request Apr 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bors Someone has bors r+ this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants