Skip to content
This repository has been archived by the owner on Nov 27, 2023. It is now read-only.

chore(storybook): re-enable for pull requests #1805

Merged
merged 3 commits into from
May 17, 2023

Conversation

kahboom
Copy link
Contributor

@kahboom kahboom commented May 16, 2023

This PR adds support for building stories on pull requests to kaoto-ui from any fork or branch, excluding Dependabot PRs.

Resolves #1100

Storybook no longer provides its own Babel configuration, so we have had to introduce it. This should not cause build issues as we are using Webpack and ts-loader for transpilation for the project.

Follow-up PR will include Storybook MSW add-on for mocking network requests.

@kahboom kahboom requested a review from a team May 16, 2023 16:17
@codecov
Copy link

codecov bot commented May 16, 2023

Codecov Report

Merging #1805 (88860bb) into main (7c855c3) will increase coverage by 0.02%.
The diff coverage is 42.85%.

❗ Current head 88860bb differs from pull request most recent head 4ba7bc3. Consider uploading reports for the commit 4ba7bc3 to get more accurate results

@@            Coverage Diff             @@
##             main    #1805      +/-   ##
==========================================
+ Coverage   65.73%   65.76%   +0.02%     
==========================================
  Files          74       74              
  Lines        2212     2211       -1     
  Branches      512      514       +2     
==========================================
  Hits         1454     1454              
+ Misses        710      709       -1     
  Partials       48       48              
Impacted Files Coverage Δ
src/services/visualizationService.ts 81.53% <ø> (ø)
src/components/PlusButtonEdge.tsx 30.55% <33.33%> (+0.82%) ⬆️
src/services/validationService.ts 84.05% <100.00%> (ø)

lordrip
lordrip previously approved these changes May 16, 2023
@@ -1 +1,2 @@
CHROMATIC_PROJECT_TOKEN=14f589a59855
Copy link
Member

Choose a reason for hiding this comment

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

Although is not ideal, we agreed on having this token shared among contributors to allow pull request checks from forks.

More info: https://www.chromatic.com/docs/github-actions#run-chromatic-on-external-forks-of-open-source-projects


const foreignObjectSize = 40;

export interface IPlusButtonEdge {
data?: any;
data?: {
Copy link
Member

Choose a reason for hiding this comment

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

[curious-question]: What should be the right type for this component? I couldn't find a type combining showBranchesTab and sourceStepNode 🤔

Copy link
Member

Choose a reason for hiding this comment

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

igarashitm
igarashitm previously approved these changes May 16, 2023
Copy link
Contributor

@igarashitm igarashitm left a comment

Choose a reason for hiding this comment

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

🎉

@kahboom kahboom dismissed stale reviews from igarashitm and lordrip via 88860bb May 17, 2023 08:09
@kahboom kahboom force-pushed the storybook-ci-cd branch from 138bff7 to 88860bb Compare May 17, 2023 08:09
kahboom added 3 commits May 17, 2023 11:39
fix(storybook): fix typescript ide errors

chore(storybook): migrate to Meta, StoryFn
…move biz logic out

fix(test): fix edge tests for updated data structure

fix: fix issue with plus button edge

update yarn.lock

minor changes

chore: rebase

chore: rebase

add storybook-static to gitignore
@kahboom kahboom force-pushed the storybook-ci-cd branch from 88860bb to 4ba7bc3 Compare May 17, 2023 10:39
@sonarcloud
Copy link

sonarcloud bot commented May 17, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
4.1% 4.1% Duplication

@kahboom
Copy link
Contributor Author

kahboom commented May 17, 2023

E2E tests are being flaky again, particularly for Chrome and Edge :(

@kahboom kahboom merged commit 2d997be into KaotoIO:main May 17, 2023
@kahboom kahboom deleted the storybook-ci-cd branch May 17, 2023 13:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Re-enable Storybook
3 participants