Skip to content
This repository was archived by the owner on May 1, 2025. It is now read-only.

docs: Add a section about release management#164

Merged
tzhelyazkova merged 2 commits into
masterfrom
release_management
Aug 21, 2020
Merged

docs: Add a section about release management#164
tzhelyazkova merged 2 commits into
masterfrom
release_management

Conversation

@tzhelyazkova
Copy link
Copy Markdown
Contributor

Rename folders and files in order to list the new file before
the "Decisions and ADRs" section.

Rename folders and files in order to list the new file before
the "Decisions and ADRs" section.
@tzhelyazkova tzhelyazkova changed the title Add a section about release management docs: Add a section about release management Aug 20, 2020
Copy link
Copy Markdown
Member

@itamargiv itamargiv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 💯


## Releasing changes to an existing component or tokens

Design System changes are supposed to influence application styles, therefore a Design System change is not finished until it's propagated to all consuming applications.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it make sense to link to https://wmde.github.io/wikit/?path=/story/documentation-how-to-modify-existing-components--page from here? Not sure if you would prefer to make any part of this sentence a hyperlink, or add a "(See xx)" at the end.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure, depends on what that page introduces. I guess we can merge this and when #165 is merged we can ammend this to link to it.

@tzhelyazkova tzhelyazkova merged commit df3854a into master Aug 21, 2020
@tzhelyazkova tzhelyazkova deleted the release_management branch August 21, 2020 11:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants