-
Notifications
You must be signed in to change notification settings - Fork 16
Remove TL Role from implementation and documentation process description #226
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
Conversation
|
The created documentation from the pull request is available at: docu-html |
masc2023
left a comment
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.
Looks good, but let's wait until official decision here eclipse-score/score#1624 (comment)
pahmann
left a comment
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 heading of the PR is incorrect as it assumes TL role is removed from process description, but it only removes it from two process areas as the linked tickets indicate.
| * contributing and approving documents | ||
|
|
||
| #. :need:`Technical Lead <rl__technical_lead>` | ||
| #. :need:`Technical Lead <rl__project_lead>` |
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.
| #. :need:`Technical Lead <rl__project_lead>` | |
| #. :need:`Project Lead <rl__project_lead>` |
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.
Changed it and also change the heading of the PR
|
PR not yet ready to be merged based on comment from @masc2023 |
Resolves:
#215
#225