add platform team definition of done#59993
Conversation
|
Pinging @elastic/kibana-platform (Team:Platform) |
pgayvallet
left a comment
There was a problem hiding this comment.
-
Is that a DOD for
coreor for all our codeowned code? -
Should we add any of thoses?
- All the impacted code is in Typescript (mostly thinking about migrating features here, so not sure if relevant)
- The feature's code and structure follows our code/naming conventions
| ## Organisational Conventions | ||
| ### Definition of done | ||
| Definition of done for a feature: | ||
| - has a dedicated Github issue describing problem space |
There was a problem hiding this comment.
Isn't that more in the 'Definition of Ready'?
There was a problem hiding this comment.
Agree, do we want to have a separate document? We can keep it here for the time being.
There was a problem hiding this comment.
Keeping it there for now is fine.
src/core/CONVENTIONS.md
Outdated
| - for a logged-in user | ||
| - for anonymous user | ||
| - compatible with Spaces | ||
| - can be used by the first & third party plugins |
That's a high-level convention applicable to all our code.
We can state it, as well a declare our approach to adding new / evaluating existing dependencies.
|
joshdover
left a comment
There was a problem hiding this comment.
A couple more suggestions:
- has appropriate logging on the server-side
- has been verified manually by at least one reviewer
that sounds confusing. we don't have a definition of
👍 |
joshdover
left a comment
There was a problem hiding this comment.
that sounds confusing. we don't have a definition of
appropriate logging😅 I think we should develop recommendations (what, when, how) for platform & plugins, but it's a separate task.
👍
joshdover
left a comment
There was a problem hiding this comment.
Let's go ahead and get this in
* add platform team definition of done * add link to ToC * add suggestions from Josh * add tooling * add testing note
* master: (56 commits) [i18n] Update CODEOWNERS (elastic#63354) add platform team definition of done (elastic#59993) [SIEM] move away from Joi for importing/exporting timeline (elastic#62125) Fix discover preserve url (elastic#63580) [alerting] Adds an alertServices mock and uses it in siem, monitoring and uptime (elastic#63489) Closes elastic#63109 for Service Map by resetting edges styles for the selected node (elastic#63655) MIgrated index_header to react (elastic#63490) Index pattern management UI -> TypeScript and New Platform Ready (indexed_fields_table) (elastic#63364) [SIEM] [Cases] Insert timeline and reporters/tags in table bug fixes (elastic#63642) [Reporting] Make usable default element positions (elastic#63191) [Reporting] Switch Serverside Config Wrapper to NP (elastic#62500) [Reporting] Add "warning" status as an alternate type of completed job (elastic#63498) Split action types into own page (elastic#63516) [Lens] Only show copy on save for previously saved docs (elastic#63535) Update README.md (elastic#63622) Bugfix clear saved query crashes kibana on Discover in some cases (elastic#63554) Add uptime CODEOWNER entries. (elastic#63616) [ML] Extract apiDoc params from the schema definitions (elastic#62933) Fix alerting documentation encryption key requirement (elastic#63512) Fix CODEOWNERS and sass lint paths (elastic#63552) ...
[skip-ci]
A draft of
Definition of donefor the platform team. Feedback and improvements are welcome.Unfortunately, it's not so easy to add several PR templates for the repo, so having the document is just fine.