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

Clarify RULES #103

Merged
merged 4 commits into from
Sep 17, 2021
Merged

Clarify RULES #103

merged 4 commits into from
Sep 17, 2021

Conversation

erget
Copy link
Member

@erget erget commented May 13, 2020

Fixes #102.

@ethanrd
Copy link
Member

ethanrd commented May 13, 2020

@erget - I believe the @cf-convention/info-mgmt team is only visible to members of the cf-convention organization. The GitHub Change Team Visibility page lists "visible" (to organization members) and "secret" (only team members). I'm not seeing a way to make it more broadly visible. So, perhaps reference the list on the governance.md page instead of the team?

@ethanrd

This comment has been minimized.

@DocOtak

This comment has been minimized.

@nigelbritton

This comment has been minimized.

@erget

This comment has been minimized.

@ethanrd

This comment has been minimized.

@erget

This comment has been minimized.

@ethanrd

This comment has been minimized.

@erget erget force-pushed the master branch 2 times, most recently from 72e1418 to 3c014de Compare May 18, 2020 16:03
ethanrd added a commit to ethanrd/cf-convention.github.io that referenced this pull request May 22, 2020
DocOtak added a commit that referenced this pull request May 22, 2020
Fix GHA for testing PRs (see discussion in #103)
@erget

This comment has been minimized.

@DocOtak

This comment has been minimized.

@erget

This comment has been minimized.

@erget

This comment has been minimized.

@erget

This comment has been minimized.

@ethanrd

This comment has been minimized.

@erget erget marked this pull request as ready for review May 27, 2020 15:36
@erget erget changed the title WIP: Add initial CONTRIBUTING.md Add initial CONTRIBUTING.md May 27, 2020
@erget erget marked this pull request as draft June 19, 2020 08:00
@erget erget changed the title Add initial CONTRIBUTING.md Add initial CONTRIBUTING.md & clarify RULES Jun 22, 2020
@erget erget mentioned this pull request Jun 22, 2020
erget added 3 commits June 22, 2020 11:11
Redundant information is removed but otherwise the only changes are
copyedit in order to make the sentences shorter and thus easier to
parse.

Other changes:
- Add diagram to describe change rules
- Build diagram when changed and embed in rules
- Integrate content from 'errors' into 'rules'
- General copyedit and refactoring of sentences onto single lines
@erget erget changed the title Add initial CONTRIBUTING.md & clarify RULES Clarify RULES Oct 16, 2020
@erget
Copy link
Member Author

erget commented Jan 14, 2021

I seem not to be able to change the from branch but I've backed this up to 103-clarify-rules now. Shoulda used a topic branch from the get-go.

@erget erget merged commit 9ec5cac into cf-convention:master Sep 17, 2021
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.

Refactor "rules"
4 participants