Skip to content

Fix the punycode deprecated message from the source (survives building from eslint.config.ts). #460

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

Merged
merged 1 commit into from
Apr 14, 2025

Conversation

Heyian
Copy link
Contributor

@Heyian Heyian commented Apr 12, 2025

This fixes the issue #426 in a way that it survives a new build from esbuild.config.js

feat(package.json): add overrides for ajv and whatwg-url packages
chore(deps): update eslint and typescript-eslint dependencies
fix(deepseek.ts): correct import path for OpenAiEngine and OpenAiConfig

The issue comes from the ajv and whatwg-url that the eslint module is using. That's why I created an override in package.json.

I've also updated the eslint module, initially I thought that it would fix the issue but even in the latest version, it's still using old version of ajv and whatwg-url. I kept those module updated since I couldn't find any issues with them.

I've also made a simple correction to the deepseek engine, there was a typo in the OpenAi module.

At first I didn't include what was in the /out folder but I couldn't find any Github actions that runs node esbuild.config.js, I might be totally wrong here, so if you have something that builds automatically on push on the master branch, I suggest I correct this PR to remove those files.

feat(package.json): add overrides for ajv and whatwg-url packages
fix(deepseek.ts): correct import path for OpenAiEngine and OpenAiConfig
@di-sukharev di-sukharev merged commit 22f96b3 into di-sukharev:master Apr 14, 2025
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