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

Update prismTheme.js export to use CommonJS syntax #9

Merged

Conversation

rossdanlm
Copy link
Contributor

@rossdanlm rossdanlm commented Oct 2, 2024

Tried running yarn build but it didn't work. I asked Claude for help and it told me I had to update this line and then it worked. I tried looking at how we did it with the docs/eval-docs part of lmai repo (https://github.com/lastmile-ai/lmai/blob/main/docs/eval-docs/tsconfig.json), but couldn't get it to work with those changes.

This gets me unblocked so doing it for now and putting in really easy line so that it's easy to review

Test Plan

yarn build

Copy link
Contributor Author

rossdanlm commented Oct 2, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rossdanlm and the rest of your teammates on Graphite Graphite

@rossdanlm rossdanlm requested a review from rholinshead October 2, 2024 19:54
@rossdanlm rossdanlm marked this pull request as ready for review October 2, 2024 19:54
@rossdanlm rossdanlm changed the base branch from really_simple_updates_to_README_list to graphite-base/9 October 2, 2024 19:55
@rossdanlm rossdanlm force-pushed the 10-02-update_prismtheme.js_export_to_use_commonjs_syntax branch from ffaed2f to 4c85d9c Compare October 2, 2024 19:55
@rossdanlm rossdanlm changed the base branch from graphite-base/9 to main October 2, 2024 19:55
@rossdanlm rossdanlm force-pushed the 10-02-update_prismtheme.js_export_to_use_commonjs_syntax branch from 4c85d9c to 05585d0 Compare October 2, 2024 19:55
@rossdanlm rossdanlm merged commit 24b6578 into main Oct 2, 2024
2 checks passed
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