Skip to content

Add ES2017 build target - #533

Open
thinklanguages wants to merge 1 commit into
mainfrom
codex/update-vite.config.js-target-option
Open

Add ES2017 build target#533
thinklanguages wants to merge 1 commit into
mainfrom
codex/update-vite.config.js-target-option

Conversation

@thinklanguages

@thinklanguages thinklanguages commented Jun 28, 2025

Copy link
Copy Markdown
Owner

Summary

  • target older browsers by specifying target: 'es2017' in vite.config.js

Testing

  • npm run build (fails: vite: not found)

https://chatgpt.com/codex/tasks/task_e_685f7d7d65e0832785a67c64e7716a22

Description by Korbit AI

What change is being made?

Add ES2017 build target to the Vite configuration.

Why are these changes being made?

To utilize modern JavaScript features introduced in ES2017, providing better performance and features for supported browsers while maintaining compatibility where needed. This aligns the project with newer JavaScript standards, optimizing the build process and potential future integrations.

Is this description stale? Ask me to generate a new description by commenting /korbit-generate-pr-description

@korbit-ai korbit-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I've completed my review and didn't find any issues... but I did find this kitten.

    |\__/,|   (`\
  _.|o o  |_   ) )
-(((---(((--------
Files scanned
File Path Reviewed
vite.config.js

Explore our documentation to understand the languages and file types we support and the files we ignore.

Check out our docs on how you can make Korbit work best for you and your team.

Loving Korbit!? Share us on LinkedIn Reddit and X

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