-
Notifications
You must be signed in to change notification settings - Fork 0
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 MAINTAINERS.md #60
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Joris Mancini <53527338+TheMaskedTurtle@users.noreply.github.com>
Signed-off-by: Joris Mancini <53527338+TheMaskedTurtle@users.noreply.github.com>
**Reviewers:** [geofjamg](https://github.com/geofjamg), [jonenst](https://github.com/jonenst), [antoinebhs](https://github.com/antoinebhs) | ||
**Committers:** [geofjamg](https://github.com/geofjamg), [jonenst](https://github.com/jonenst), [antoinebhs](https://github.com/antoinebhs) | ||
**Reviewers:** [jonenst](https://github.com/jonenst), [antoinebhs](https://github.com/antoinebhs), [EtienneLt](https://github.com/EtienneLt), [SlimaneAmar](https://github.com/SlimaneAmar), [FranckLecuyer](https://github.com/FranckLecuyer) | ||
**Committers:** [jonenst](https://github.com/jonenst), [antoinebhs](https://github.com/antoinebhs), [SlimaneAmar](https://github.com/SlimaneAmar), [FranckLecuyer](https://github.com/FranckLecuyer) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to keep it to Jon and Antoine? They are already committers on other repos so we do not add more committers if we stick to them only.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We decided at project level that these four people are the new responsibles of the network-store topics as experts on the subject, we are all split in teams now in GridSuite so it would be easier if they could all be commiters
This [repository](https://github.com/powsybl/powsybl-network-store-server) provides a client for the persistent implementation of the IIDM grid model. | ||
|
||
**Reviewers:** [jonenst](https://github.com/jonenst), [antoinebhs](https://github.com/antoinebhs), [EtienneLt](https://github.com/EtienneLt), [SlimaneAmar](https://github.com/SlimaneAmar), [FranckLecuyer](https://github.com/FranckLecuyer) | ||
**Committers:** [jonenst](https://github.com/jonenst), [antoinebhs](https://github.com/antoinebhs), [SlimaneAmar](https://github.com/SlimaneAmar), [FranckLecuyer](https://github.com/FranckLecuyer) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same remark, do you need 2 new committers?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same answer
**Reviewers:** [geofjamg](https://github.com/geofjamg), [jonenst](https://github.com/jonenst) | ||
**Committers:** [geofjamg](https://github.com/geofjamg), [jonenst](https://github.com/jonenst) | ||
**Reviewers:** [SlimaneAmar](https://github.com/SlimaneAmar), [antoinebhs](https://github.com/antoinebhs) | ||
**Committers:** [SlimaneAmar](https://github.com/SlimaneAmar), [antoinebhs](https://github.com/antoinebhs) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same remark as above. Could you justify the need to add a new committer?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The team in charge of the related topics in GridSuite project is the team B where Slimane is the tech lead and where Antoine can second him. Geoffroy and Jon are not specifically related to these topics anymore
MAINTAINERS.md
Outdated
**Reviewers:** [flo-dup](https://github.com/flo-dup) | ||
**Committers:** [flo-dup](https://github.com/flo-dup) | ||
**Reviewers:** [SlimaneAmar](https://github.com/SlimaneAmar), [antoinebhs](https://github.com/antoinebhs) | ||
**Committers:** [SlimaneAmar](https://github.com/SlimaneAmar), [antoinebhs](https://github.com/antoinebhs) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same remark as above.
MAINTAINERS.md
Outdated
|
||
### powsybl-diagram-viewer | ||
This [repository](https://github.com/powsybl/powsybl-diagram-viewer) provides a service to display network-area digrams and single-line diagrams. | ||
|
||
**Reviewers:** [flo-dup](https://github.com/flo-dup) | ||
**Committers:** [flo-dup](https://github.com/flo-dup) | ||
**Reviewers:** [SlimaneAmar](https://github.com/SlimaneAmar), [antoinebhs](https://github.com/antoinebhs) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@flo-dup should be left as a reviewer and a committer. He works on a regular basis with this repository.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok I asked in my message in the channel but nobody answered ^^ I'll add him back
MAINTAINERS.md
Outdated
**Reviewers:** [jonenst](https://github.com/jonenst) | ||
**Committers:** [jonenst](https://github.com/jonenst) | ||
**Reviewers:** [jonenst](https://github.com/jonenst), [TheMaskedTurtle](https://github.com/TheMaskedTurtle), [SlimaneAmar](https://github.com/SlimaneAmar), [etiennehomer](https://github.com/etiennehomer), [antoinebhs](https://github.com/antoinebhs), [EtienneLt](https://github.com/EtienneLt), [AbdelHedhili](https://github.com/AbdelHedhili), [FranckLecuyer](https://github.com/FranckLecuyer) | ||
**Committers:** [jonenst](https://github.com/jonenst), [TheMaskedTurtle](https://github.com/TheMaskedTurtle), [SlimaneAmar](https://github.com/SlimaneAmar), [etiennehomer](https://github.com/etiennehomer), [antoinebhs](https://github.com/antoinebhs), [EtienneLt](https://github.com/EtienneLt), [AbdelHedhili](https://github.com/AbdelHedhili), [FranckLecuyer](https://github.com/FranckLecuyer) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are a lot of people here! We should keep the number of committers to a minimum.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will reduce it
MAINTAINERS.md
Outdated
This [repository](https://github.com/powsybl/powsybl-ws-dependencies) provides dependency versions for powsybl web services. | ||
|
||
**Reviewers:** [jonenst](https://github.com/jonenst), [TheMaskedTurtle](https://github.com/TheMaskedTurtle), [SlimaneAmar](https://github.com/SlimaneAmar), [etiennehomer](https://github.com/etiennehomer), [antoinebhs](https://github.com/antoinebhs), [EtienneLt](https://github.com/EtienneLt), [AbdelHedhili](https://github.com/AbdelHedhili), [FranckLecuyer](https://github.com/FranckLecuyer) | ||
**Committers:** [jonenst](https://github.com/jonenst), [TheMaskedTurtle](https://github.com/TheMaskedTurtle), [SlimaneAmar](https://github.com/SlimaneAmar), [etiennehomer](https://github.com/etiennehomer), [antoinebhs](https://github.com/antoinebhs), [EtienneLt](https://github.com/EtienneLt), [AbdelHedhili](https://github.com/AbdelHedhili), [FranckLecuyer](https://github.com/FranckLecuyer) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same remark, this is really a lot. Is it necessary?
Signed-off-by: Joris Mancini <53527338+TheMaskedTurtle@users.noreply.github.com>
Please check if the PR fulfills these requirements
What kind of change does this PR introduce?
Change list of maintainers for ws repositories