Skip to content

Telemetry: Add query-parameter to README.md#32283

Merged
shilman merged 11 commits into
nextfrom
shilman/readme-utm-params
Aug 19, 2025
Merged

Telemetry: Add query-parameter to README.md#32283
shilman merged 11 commits into
nextfrom
shilman/readme-utm-params

Conversation

@shilman
Copy link
Copy Markdown
Member

@shilman shilman commented Aug 17, 2025

Closes N/A

What I did

  • Update README files to with tracking parameters (ref=readme) to Storybook links.
  • Add a "Learn more about Storybook" paragraph with a direct link to the Storybook homepage.
  • Restore ESLint Plugin README that was lost when the Plugin was moved to the monorepo

🦋 Canary release

This PR does not have a canary release associated. You can request a canary release of this pull request by mentioning the @storybookjs/core team here.

core team members can create a canary release here or locally with gh workflow run --repo storybookjs/storybook canary-release-pr.yml --field pr=<PR_NUMBER>

@shilman shilman added documentation patch:yes Bugfix & documentation PR that need to be picked to main branch ci:docs Run the CI jobs for documentation checks only. labels Aug 17, 2025
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

43 files reviewed, 6 comments

Edit Code Review Bot Settings | Greptile

Comment thread code/frameworks/preact-vite/README.md Outdated
Comment thread code/addons/themes/README.md Outdated
Comment thread code/lib/eslint-plugin/README.md Outdated
export default [
// ...
{
// Inside your .eslintignore file
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.

logic: Comment refers to .eslintignore file but this is actually an eslint.config.js flat config example

Suggested change
// Inside your .eslintignore file
// Inside your eslint.config.js file

Comment thread code/frameworks/server-webpack5/README.md Outdated
Comment thread code/presets/server-webpack/README.md Outdated
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
@nx-cloud
Copy link
Copy Markdown

nx-cloud Bot commented Aug 17, 2025

View your CI Pipeline Execution ↗ for commit e62f151

Command Status Duration Result
nx run-many -t build --parallel=3 ✅ Succeeded 47s View ↗

☁️ Nx Cloud last updated this comment at 2025-08-19 02:05:55 UTC

shilman and others added 5 commits August 17, 2025 17:44
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
@shilman shilman self-assigned this Aug 17, 2025
@shilman shilman changed the title Documentation: Add README UTM parameter Documentation: Add ref=readme parameter to README.md Aug 19, 2025
@shilman shilman merged commit 6519472 into next Aug 19, 2025
9 checks passed
@shilman shilman deleted the shilman/readme-utm-params branch August 19, 2025 01:50
@ndelangen ndelangen changed the title Documentation: Add ref=readme parameter to README.md Telemetry: Add query-parameter to README.md Aug 20, 2025
ndelangen pushed a commit that referenced this pull request Aug 20, 2025
Documentation: Add ref=readme parameter to README.md
(cherry picked from commit 6519472)
ndelangen pushed a commit that referenced this pull request Aug 21, 2025
Documentation: Add ref=readme parameter to README.md
(cherry picked from commit 6519472)
@github-actions github-actions Bot added the patch:done Patch/release PRs already cherry-picked to main/release branch label Aug 21, 2025
@ndelangen ndelangen removed the patch:yes Bugfix & documentation PR that need to be picked to main branch label Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:docs Run the CI jobs for documentation checks only. documentation patch:done Patch/release PRs already cherry-picked to main/release branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants