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

Further simplify principles #12

Conversation

scottrigby
Copy link
Member

@scottrigby scottrigby commented Jul 3, 2021

🚧 Important note: this is not a PR against main, only against the PR branch for #9. Therefore it doesn't need to pass all scrutiny to merge, only that the owner of #9 (@todaywasawesome) is comfortable merging this into their PR, which still needs the Principles Committee to sign off on.

@todaywasawesome I could have pushed these changes directly to your PR branch, but I want to make sure you feel comfortable with these since that's your PR and I took some liberty in my suggested changes here. I think the changes are good, and I'm interested to hear your thoughts and discuss.

This PR also proposes new glossary item solutions to addressing #11, as another step toward the v1.0.0 milestone. Commits are separated, see commit messages for details.

- Simplify first line of each principle, for easy memorization.
- Swap principles 3 and 4. I think this is what we ultimately wanted to convey.
- Swap notes for principles 3 and 4 to match.
- Move principle 4 notes to Glossary (Drift and Reconciliation)

Signed-off-by: Scott Rigby <[email protected]>
…ent of that forthcoming recipe while not assuming Git

Signed-off-by: Scott Rigby <[email protected]>
Signed-off-by: Scott Rigby <[email protected]>
…s is the gist of the note, including the to-do at the end

Signed-off-by: Scott Rigby <[email protected]>
@scottrigby scottrigby changed the title Simplify principles Further simplify principles Jul 7, 2021
@scottrigby
Copy link
Member Author

After today's meeting this PR contains too many changes. Closing in favor of #14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant