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

Maintenance updates #87

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Maintenance updates #87

wants to merge 2 commits into from

Conversation

EPinci
Copy link

@EPinci EPinci commented May 18, 2023

  • Updated all base dependencies by resetting to a current starter extension template
    • Grabbed a current extension starter template (note the new dependencies and devDependencies)
    • Copied over the /src folder
  • Updated code dependencies to latest available versions
  • Consistently applied linting (now working without extensions) fixes thru the code
    • Exception for "Require statement not part of import statement @typescript-eslint/no-var-requires" that I couldn't find a way to fix
  • Consistently removed deprecated calls thru the code
  • Removed test stubs since there are no tests (test stubs are not anymore in the starter extension template)

Note that the @vscode/vsce dependency is now supposed to be installed globally to provide the vsce binary

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.

1 participant