Skip to content
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

Fixes for storybook aliases #204842

Merged
merged 11 commits into from
Dec 26, 2024
Merged

Fixes for storybook aliases #204842

merged 11 commits into from
Dec 26, 2024

Conversation

smith
Copy link
Contributor

@smith smith commented Dec 18, 2024

  • Remove duplicates
  • Import list of aliases to buildkite script so we don't need it in two places
  • Mention packages as well as plugins in README, and use newer example storybook syntax
  • Add mjs support to unbreak slo stories

* Remove duplicates
* Import list of aliases to buildkite script so we don't need it in two places
* Mention packages as well as plugins in README, and use newer example storybook syntax
@smith smith marked this pull request as ready for review December 19, 2024 11:53
@smith smith requested a review from a team as a code owner December 19, 2024 11:53
@smith smith added backport:prev-major Backport to (8.x, 8.17, 8.16) the previous major branch and other branches in development release_note:skip Skip the PR/issue when compiling release notes labels Dec 19, 2024
Copy link
Contributor

@Ikuni17 Ikuni17 left a comment

Choose a reason for hiding this comment

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

Thanks for putting this PR together. I think this is good, waiting for Build Storybooks step to pass for approval

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

@smith smith requested a review from Ikuni17 December 25, 2024 14:18
@smith smith enabled auto-merge (squash) December 25, 2024 14:18
@smith smith merged commit ddcf076 into elastic:main Dec 26, 2024
9 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.16, 8.17, 8.x

https://github.com/elastic/kibana/actions/runs/12506166464

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.16 Backport failed because of merge conflicts
8.17 Backport failed because of merge conflicts
8.x Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 204842

Questions ?

Please refer to the Backport tool documentation

smith added a commit to smith/kibana that referenced this pull request Dec 27, 2024
* Remove duplicates
* Import list of aliases to buildkite script so we don't need it in two
places
* Mention packages as well as plugins in README, and use newer example
storybook syntax
* Add mjs support to unbreak slo stories

(cherry picked from commit ddcf076)

# Conflicts:
#	.buildkite/scripts/steps/storybooks/build_and_upload.ts
#	src/dev/storybook/aliases.ts
@smith
Copy link
Contributor Author

smith commented Dec 27, 2024

💚 All backports created successfully

Status Branch Result
8.x
8.17
8.16

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

smith added a commit to smith/kibana that referenced this pull request Dec 27, 2024
* Remove duplicates
* Import list of aliases to buildkite script so we don't need it in two
places
* Mention packages as well as plugins in README, and use newer example
storybook syntax
* Add mjs support to unbreak slo stories

(cherry picked from commit ddcf076)
smith added a commit to smith/kibana that referenced this pull request Dec 27, 2024
* Remove duplicates
* Import list of aliases to buildkite script so we don't need it in two
places
* Mention packages as well as plugins in README, and use newer example
storybook syntax
* Add mjs support to unbreak slo stories

(cherry picked from commit ddcf076)

# Conflicts:
#	.buildkite/scripts/steps/storybooks/build_and_upload.ts
#	src/dev/storybook/aliases.ts
@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Dec 27, 2024
@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

smith added a commit that referenced this pull request Dec 30, 2024
# Backport

This will backport the following commits from `main` to `8.x`:
- [Fixes for storybook aliases
(#204842)](#204842)

<!--- Backport version: 8.9.8 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Nathan L
Smith","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-12-26T16:38:22Z","message":"Fixes
for storybook aliases (#204842)\n\n* Remove duplicates\r\n* Import list
of aliases to buildkite script so we don't need it in two\r\nplaces\r\n*
Mention packages as well as plugins in README, and use newer
example\r\nstorybook syntax\r\n* Add mjs support to unbreak slo
stories","sha":"ddcf076ff400315a66e6076d694b301cf7d84070","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-major","ci:build-storybooks"],"number":204842,"url":"https://github.com/elastic/kibana/pull/204842","mergeCommit":{"message":"Fixes
for storybook aliases (#204842)\n\n* Remove duplicates\r\n* Import list
of aliases to buildkite script so we don't need it in two\r\nplaces\r\n*
Mention packages as well as plugins in README, and use newer
example\r\nstorybook syntax\r\n* Add mjs support to unbreak slo
stories","sha":"ddcf076ff400315a66e6076d694b301cf7d84070"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/204842","number":204842,"mergeCommit":{"message":"Fixes
for storybook aliases (#204842)\n\n* Remove duplicates\r\n* Import list
of aliases to buildkite script so we don't need it in two\r\nplaces\r\n*
Mention packages as well as plugins in README, and use newer
example\r\nstorybook syntax\r\n* Add mjs support to unbreak slo
stories","sha":"ddcf076ff400315a66e6076d694b301cf7d84070"}}]}]
BACKPORT-->
@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

1 similar comment
@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

stratoula pushed a commit to stratoula/kibana that referenced this pull request Jan 2, 2025
* Remove duplicates
* Import list of aliases to buildkite script so we don't need it in two
places
* Mention packages as well as plugins in README, and use newer example
storybook syntax
* Add mjs support to unbreak slo stories
benakansara pushed a commit to benakansara/kibana that referenced this pull request Jan 2, 2025
* Remove duplicates
* Import list of aliases to buildkite script so we don't need it in two
places
* Mention packages as well as plugins in README, and use newer example
storybook syntax
* Add mjs support to unbreak slo stories
@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport missing Added to PRs automatically when the are determined to be missing a backport. backport:prev-major Backport to (8.x, 8.17, 8.16) the previous major branch and other branches in development ci:build-storybooks release_note:skip Skip the PR/issue when compiling release notes v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants