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

Upgrade TypeScript and Svelte Versions #995

Merged
merged 5 commits into from
Jul 16, 2024
Merged

Upgrade TypeScript and Svelte Versions #995

merged 5 commits into from
Jul 16, 2024

Conversation

clysto
Copy link
Contributor

@clysto clysto commented Jul 16, 2024

Description

This PR upgrades the TypeScript and Svelte versions used in the Warpgate project. The specific changes include:

  1. TypeScript Upgrade

    • Upgraded TypeScript version from 4.9.4 to 5.0.0 in package.json.
    • Updated related TypeScript dependencies in package.json.
  2. Svelte Upgrade

    • Upgraded Svelte version from 3.58.0 to 4.0.0 in package.json.
    • Updated svelte-eslint-parser settings in .eslintrc.yaml.
    • Replaced sveltestrap imports with @sveltestrap/sveltestrap across various Svelte components.
  3. ESLint Configuration

    • Added plugin:svelte/recommended to .eslintrc.yaml.
    • Updated eslint and related plugins to compatible versions.
  4. Other Dependencies

    • Updated several other dependencies in package.json and yarn.lock to their latest versions to ensure compatibility with TypeScript 5 and Svelte 4.

Changes Summary

  • .eslintrc.yaml: Adjusted ESLint configurations for the new Svelte parser and updated TypeScript rules.
  • openapitools.json: Upgraded openapi-generator-cli version to 7.7.0.
  • package.json: Updated versions of TypeScript, Svelte, ESLint, and other dependencies.
  • Various .svelte files: Replaced sveltestrap imports with @sveltestrap/sveltestrap and made minor type adjustments for TypeScript 5 compatibility.
  • tsconfig.json: Minor configuration adjustment to support new TypeScript features.

Copy link
Member

@Eugeny Eugeny left a comment

Choose a reason for hiding this comment

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

Thank you for contributing! I've left a few comments, otherwise LGTM. Please also merge the latest main brain to fix the builds.

warpgate-web/src/admin/Target.svelte Outdated Show resolved Hide resolved
warpgate-web/src/admin/User.svelte Outdated Show resolved Hide resolved
warpgate-web/src/common/CopyButton.svelte Show resolved Hide resolved
@Eugeny Eugeny merged commit b65a189 into warp-tech:main Jul 16, 2024
6 of 7 checks passed
@Eugeny
Copy link
Member

Eugeny commented Jul 16, 2024

Perfect, thank you!

@all-contributors please add @clysto for code

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