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

fix: update deploy docs for storybook 7.x #1733

Merged
merged 3 commits into from
Aug 23, 2023

Conversation

booc0mtaco
Copy link
Contributor

@booc0mtaco booc0mtaco commented Aug 23, 2023

Summary:

Ref: storybookjs/storybook#20932

  • move away from deprecated deploy package and instead rely directly on GH Actions
  • remove dependency
  • update package.json scripts
  • test deploy to gh-pages then revert branch to read

Test Plan:

@booc0mtaco booc0mtaco force-pushed the aholloway/x-fix-gh-pages-deploy branch from e583c5e to 010d91b Compare August 23, 2023 18:35
@github-actions
Copy link

github-actions bot commented Aug 23, 2023

size-limit report 📦

Path Size
components 95.27 KB (0%)
styles 32.68 KB (0%)

@codecov
Copy link

codecov bot commented Aug 23, 2023

Codecov Report

Merging #1733 (d80ed06) into next (345bdbf) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             next    #1733      +/-   ##
==========================================
- Coverage   92.32%   92.28%   -0.04%     
==========================================
  Files         146      146              
  Lines        2579     2579              
  Branches      665      665              
==========================================
- Hits         2381     2380       -1     
  Misses        183      183              
- Partials       15       16       +1     

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@booc0mtaco booc0mtaco force-pushed the aholloway/x-fix-gh-pages-deploy branch 5 times, most recently from a8e098d to a8d65d5 Compare August 23, 2023 19:06
@booc0mtaco booc0mtaco changed the base branch from main to next August 23, 2023 20:01
@booc0mtaco booc0mtaco marked this pull request as ready for review August 23, 2023 20:02
@booc0mtaco booc0mtaco requested a review from a team August 23, 2023 20:04
@booc0mtaco
Copy link
Contributor Author

booc0mtaco commented Aug 23, 2023

Note: completed publish of v13.1.0 since this fix also did a docs deploy of what's on next ... which as a result of the release is the same as documenting what WAS released. no need to immediately do another release

- name: Deploy Storybook
uses: peaceiris/actions-gh-pages@v3
with:
publish_dir: ./storybook-static
Copy link
Contributor Author

Choose a reason for hiding this comment

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

the naming is a bit awkward here... this should be read as dir_to_publish or similar; this isn't where the docs will be published to, and won't end up in the path

@booc0mtaco booc0mtaco force-pushed the aholloway/x-fix-gh-pages-deploy branch from 788b248 to d80ed06 Compare August 23, 2023 20:35
@booc0mtaco booc0mtaco merged commit b3a091e into next Aug 23, 2023
8 checks passed
@booc0mtaco booc0mtaco deleted the aholloway/x-fix-gh-pages-deploy branch August 23, 2023 20:50
@booc0mtaco booc0mtaco mentioned this pull request Sep 1, 2023
booc0mtaco added a commit that referenced this pull request Sep 5, 2023
### [13.1.1](v13.1.0...v13.1.1) (2023-09-01)


### Bug Fixes

* generate valid tab ids ([#1737](#1737)) ([d1493ea](d1493ea))
* update deploy docs for storybook 7.x ([#1733](#1733)) ([b3a091e](b3a091e))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants