Skip to content

Simplify Plotly testing setup#16

Merged
cotedav merged 1 commit intomainfrom
codex/check-status-of-story-e3-f5-s1
Dec 3, 2025
Merged

Simplify Plotly testing setup#16
cotedav merged 1 commit intomainfrom
codex/check-status-of-story-e3-f5-s1

Conversation

@cotedav
Copy link
Copy Markdown
Owner

@cotedav cotedav commented Dec 3, 2025

Summary

  • remove the Vitest alias for plotly.js-dist-min so tests resolve the installed package while using the global mock
  • drop the redundant Plotly stub file that was only used by the alias

Testing

  • npm run ci

Codex Task

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@cotedav cotedav merged commit 868c425 into main Dec 3, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant