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

Update CODEOWNERS (again) #1280

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

mrossinek
Copy link
Member

Summary

After reviewing the members of the qiskit-nature Github organization team, the only member without explicit other permissions in this repo was @stefan-woerner. Based on an offline discussion he agreed that his ownership and permissions are no longer required which is why I am removing him from this file. Subsequently, I will also remove the qiskit-nature team from the repository settings since we can rather add individual members in the future (rather than first adding everyone to the qiskit-community Github organization and subsequently a single team; since we will have less overlapping members across multiple repositories).

Details and comments

After reviewing the members of the `qiskit-nature` Github organization
team, the only member without explicit other permissions in this repo
was @stefan-woerner. Based on an offline discussion he agreed that his
ownership and permissions are no longer required which is why I am
removing him from this file. Subsequently, I will also remove the
`qiskit-nature` team from the repository settings since we can rather
add individual members in the future (rather than first adding everyone
to the qiskit-community Github organization and subsequently a single
team; since we will have less overlapping members across multiple
repositories).
@mrossinek mrossinek added the stable backport potential The bug might be minimal and/or import enough to be port to stable label Nov 27, 2023
@coveralls
Copy link

Pull Request Test Coverage Report for Build 7006538559

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-0.01%) to 86.768%

Files with Coverage Reduction New Missed Lines %
qiskit_nature/second_q/drivers/psi4d/psi4driver.py 1 85.12%
Totals Coverage Status
Change from base Build 7006453610: -0.01%
Covered Lines: 8767
Relevant Lines: 10104

💛 - Coveralls

@mrossinek mrossinek merged commit aec7d3d into qiskit-community:main Nov 27, 2023
@mrossinek mrossinek deleted the update_codeowners branch November 27, 2023 14:51
mergify bot pushed a commit that referenced this pull request Nov 27, 2023
After reviewing the members of the `qiskit-nature` Github organization
team, the only member without explicit other permissions in this repo
was @stefan-woerner. Based on an offline discussion he agreed that his
ownership and permissions are no longer required which is why I am
removing him from this file. Subsequently, I will also remove the
`qiskit-nature` team from the repository settings since we can rather
add individual members in the future (rather than first adding everyone
to the qiskit-community Github organization and subsequently a single
team; since we will have less overlapping members across multiple
repositories).

(cherry picked from commit aec7d3d)

# Conflicts:
#	.github/CODEOWNERS
mrossinek added a commit that referenced this pull request Nov 27, 2023
After reviewing the members of the `qiskit-nature` Github organization
team, the only member without explicit other permissions in this repo
was @stefan-woerner. Based on an offline discussion he agreed that his
ownership and permissions are no longer required which is why I am
removing him from this file. Subsequently, I will also remove the
`qiskit-nature` team from the repository settings since we can rather
add individual members in the future (rather than first adding everyone
to the qiskit-community Github organization and subsequently a single
team; since we will have less overlapping members across multiple
repositories).

(cherry picked from commit aec7d3d)

# Conflicts:
#	.github/CODEOWNERS

Co-authored-by: Max Rossmannek <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stable backport potential The bug might be minimal and/or import enough to be port to stable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants