-
Notifications
You must be signed in to change notification settings - Fork 62
Improvement: PMP - Quality Management Plan, ML2 #1305 #1920
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
base: main
Are you sure you want to change the base?
Conversation
|
The created documentation from the pull request is available at: docu-html |
The :need:`PROCESS_rl__quality_manager` provides only support to the project with consulting them to improve the | ||
quality of the project/platform product. Although the quality report is also sent to the :need:`PROCESS_rl__technical_lead`, | ||
the :need:`PROCESS_rl__quality_manager` is independent of the delivery of the product. | ||
The Quality Manager :need:`PROCESS_rl__quality_manager` provides support to the project by consulting on quality improvements |
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.
Should we not put the :need into brackets: Quality Manager (:need: ....), I see this in the complete text, makes reading more difficult
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 compared it with other documents and prefer the shortest solution with only the need itself. It good readable so we can reduce the text a little bit.
the :need:`PROCESS_rl__quality_manager` is independent of the delivery of the product. | ||
The Quality Manager :need:`PROCESS_rl__quality_manager` provides support to the project by consulting on quality improvements | ||
for the project/platform product. The Quality Manager is independent of the product delivery process. A quality report | ||
is sent to the Technical Lead :need:`PROCESS_rl__technical_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.
According to workflow TL approve the report, it is not sent to them?
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
4.1 Quality Resources | ||
"""""""""""""""""""""" | ||
A dedicated Quality Manager is defined as part of the cross functional feature teams as defined in :doc:`project_management`. The Quality Manager :need:`PROCESS_rl__quality_manager`, supported by the | ||
Dedicated Quality Managers :need:`PROCESS_rl__quality_manager` are elected by :need:`PROCESS_rl__project_lead` for the platform. |
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 think we need to rephrase these, need here also a meriocratic process, like elections of other roles, like Project Leads
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.
Add a remark for that
- <Link to WP> | ||
- <automated> | ||
|
||
- :ref:`project_documents_list` |
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.
Spot check, Trainings Path does not exist ins the document list? How to deal with that?
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 think we can decide it tomorrow. I also found that we described a training in the plan but it's not included in the work product for the moment. I guess we can delete all entries / skip it or bring the content up and also define where the folder would be so the documentation plan can also be updated.
Closes #1305 with several improvements for ML2