Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Update Codeowners for releasing. #449

Merged
merged 1 commit into from
Jan 17, 2020
Merged

Conversation

lukasholzer
Copy link
Contributor

@lukasholzer lukasholzer commented Jan 17, 2020

Pull Request


Hi, thank you for contributing to Barista with this pull request (PR).

To ensure a fast process and merging of your PR please make sure it fulfills the
coding standards and contribution guidelines.

  • A feature proposal has been provided, discussed and approved first.
  • There is a meaningful description of the issue in GitHub (Screenshots are
    often helpful).
  • If the PR introduces breaking-changes or deprecations it matches the following
    guidelines.
    • The commit message follows our commit guidelines.
    • Tests for the changes have been added (for bug fixes / features).
    • Docs have been added / updated (for bug fixes / features).

Please choose the type appropriate for the changes below:

Type of PR

Other

Checklist

  • I have read the CONTRIBUTING doc and I follow the PR guidelines
  • Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

@@ -100,7 +100,7 @@
/tools/linting/* @thomaspink @lukasholzer
/tools/linting/src/** @rowa-audil @lara-aigmueller @heartdisease

/tools/release/** @thomaspink
/tools/release/** @lukasholzer
Copy link
Collaborator

Choose a reason for hiding this comment

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

please add at least a second person. Having only one owner is potentially blocking us from moving forward and will bottleneck and silo us.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good point will add @ffriedl89 who did the refactoring

@sonarcloud
Copy link

sonarcloud bot commented Jan 17, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@lukasholzer lukasholzer merged commit f460c19 into master Jan 17, 2020
@lukasholzer lukasholzer deleted the chore/update-codeowners branch January 17, 2020 08:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants