Skip to content

Conversation

@christian-bromann
Copy link
Contributor

Couple of comments:

  • moved the "Inspired by Zalando" note to a central place so we don't need to maintain it on several pages
  • some wording changes given that we have multiple orgs
- hosted under the Sauce Labs GitHub organization.
+ hosted under a Sauce Labs managed GitHub organization.
  • Removed Legal Council from the text as the OSPO should be the central point for questions, we ourself should reach out to Legal if necessary - this also prevents that any conversations pass by the office
  • code reviews should be done by at least 1 Sauce instead of 2 - having a commit to be signed of by 2 employees seems more cumbersome than it helps, it should be possible to maintain a project with at least 2 people.
  • Community best practices like CoC are a must, not only recommended

Copy link
Member

@diemol diemol left a comment

Choose a reason for hiding this comment

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

Just a couple of comments.

@diemol diemol merged commit 9fa7e36 into dev Jul 4, 2020
@diemol diemol deleted the cb-policy-review branch July 4, 2020 13:27
github-actions bot pushed a commit that referenced this pull request Jul 4, 2020
* some review and modifications to the policies

* make code analysis tools a recommendation 9fa7e36
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.

3 participants