Skip to content

feat(ui): add GitHub link to header help menu #19

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.

Sign up for GitHub

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

Merged
blimmer merged 5 commits into from
May 6, 2026
Merged

feat(ui): add GitHub link to header help menu #19

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
d515fea
feat(ui): add GitHub link to header help menu
blimmer May 6, 2026
26c9339
ci(chromatic): auto-trigger on UI/CSS changes instead of label gate
blimmer May 6, 2026
101c48f
fix: chromatic
blimmer May 6, 2026
39b2b5f
Merge branch 'main' into fix/header-github-link
blimmer May 6, 2026
d74d1ca
perf(chromatic): exclude demo-recorder stories from CI snapshots
blimmer May 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 8 additions & 7 deletions .github/workflows/chromatic.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ name: Chromatic

on:
pull_request:
types: [opened, synchronize, labeled, closed]
types: [opened, synchronize, reopened, closed]
paths:
- 'packages/plan/**/*.tsx'
- 'packages/plan/**/*.css'
Expand All @@ -23,13 +23,12 @@ jobs:
name: Publish
runs-on: ubuntu-latest
timeout-minutes: 10
# Run if PR has 'chromatic' label AND either:
# 1. PR is open (opened, synchronize, labeled events), OR
# 2. PR was just merged to main (closed event with merged=true)
Copy link
Copy Markdown
Contributor Author

@blimmer blimmer May 6, 2026

Choose a reason for hiding this comment

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

We don't need to be as conservative with runs since chromatic is free for OSS (IIRC)

# Triggered by the workflow-level paths filter (UI/CSS changes only). Skip the
# closed event unless it's a merge to main — closed-without-merge shouldn't
# republish baselines.
if: |
contains(github.event.pull_request.labels.*.name, 'chromatic') &&
(github.event.action != 'closed' ||
(github.event.pull_request.merged == true && github.event.pull_request.base.ref == 'main'))
github.event.action != 'closed' ||
(github.event.pull_request.merged == true && github.event.pull_request.base.ref == 'main')
steps:
- name: Checkout code
uses: actions/checkout@v6
Expand All @@ -43,6 +42,8 @@ jobs:

- name: Build Storybook
run: bun run --cwd packages/plan build-storybook
env:
SKIP_DEMO_STORIES: 'true'

- name: Deploy to Chromatic
# Use CLI directly instead of chromaui/action because the action overwrites
Expand Down
4 changes: 3 additions & 1 deletion packages/plan/.storybook/main.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,9 @@
import type { StorybookConfig } from '@storybook/react-vite';

const skipDemo = process.env.SKIP_DEMO_STORIES === 'true';

const config: StorybookConfig = {
stories: ['../src/**/*.stories.@(js|jsx|mjs|ts|tsx)'],
stories: [skipDemo ? '../src/**/!(*.demo).stories.@(js|jsx|mjs|ts|tsx)' : '../src/**/*.stories.@(js|jsx|mjs|ts|tsx)'],
addons: [],
framework: '@storybook/react-vite',
};
Expand Down
Loading
Loading