-
Notifications
You must be signed in to change notification settings - Fork 393
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
fix: dependabot-library-updates #4388
Conversation
@@ -243,14 +243,14 @@ function getDialogStartingPath(): vscode.Uri | undefined { | |||
const lastOpenedLogFolder = extContext.workspaceState.get<string>( | |||
LAST_OPENED_LOG_FOLDER_KEY | |||
); | |||
if (lastOpenedLogFolder && pathExists.sync(lastOpenedLogFolder)) { | |||
if (lastOpenedLogFolder && pathExists.pathExistsSync(lastOpenedLogFolder)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With the update of the newer pathExist
library, sync()
was replaced with pathExistSync()
@@ -13,7 +13,11 @@ import { createSandbox, SinonSandbox, SinonStub } from 'sinon'; | |||
import * as vscode from 'vscode'; | |||
import { SET_JAVA_DOC_LINK } from '../../src/constants'; | |||
import { nls } from '../../src/messages'; | |||
import { checkJavaVersion, JAVA_HOME_KEY, resolveRequirements } from '../../src/requirements'; | |||
import { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Auto-formatting that took placed when I saved the file.
@@ -69,7 +73,9 @@ describe('Java Requirements Test', () => { | |||
const result = await checkJavaVersion('~/java_home'); | |||
expect(result).to.equal(true); | |||
} catch (err) { | |||
fail(`Should not have thrown when the Java version is 17. The error was: ${err}`); | |||
fail( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
more auto-formatting
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🍠 🚢
What does this PR do?
This PR updates the following libraries:
commitlint/cli
commitlint/config-conventional
glob
path-exists
types/path-exists
...and makes the following PRs redundant:
#4174
#4249
#4390