Skip to content

Conversation

@aeisenberg
Copy link
Contributor

Do not commit

@aeisenberg aeisenberg force-pushed the aeisenberg/do-not-merge branch from ae9d464 to 6966db0 Compare February 10, 2021 17:03
@adityasharad
Copy link
Contributor

Make this draft? Then you won't create notifications.

@aeisenberg aeisenberg marked this pull request as draft February 10, 2021 17:23
These functions assist with object disposal. They add custom behaviour
during disposal. The primary usage of disposalHandlers is during testing
where some objects should not be disposed in order to avoid testing
errors.
@aeisenberg aeisenberg force-pushed the aeisenberg/do-not-merge branch from 791e164 to ae748f2 Compare February 10, 2021 23:41
@aeisenberg aeisenberg force-pushed the aeisenberg/do-not-merge branch from 896b34e to fa4d031 Compare February 11, 2021 16:18
don't run any no-workspace tests. Let's see if this helps.
@aeisenberg aeisenberg force-pushed the aeisenberg/do-not-merge branch from fa4d031 to 61cb781 Compare February 11, 2021 16:26
aeisenberg added a commit to aeisenberg/vscode-codeql that referenced this pull request Feb 11, 2021
We should not be opening any external links during tests. This is
causing some builds to hang when running on CI.

See github#750 for an example.
aeisenberg added a commit to aeisenberg/vscode-codeql that referenced this pull request Feb 11, 2021
We should not be opening any external links during tests. This is
causing some builds to hang when running on CI.

See github#750 for an example.
aeisenberg added a commit that referenced this pull request Feb 11, 2021
We should not be opening any external links during tests. This is
causing some builds to hang when running on CI.

See #750 for an example.
@aeisenberg aeisenberg closed this Feb 11, 2021
aofaof0907 pushed a commit to aofaof0907/vscode-codeql that referenced this pull request Jul 27, 2021
We should not be opening any external links during tests. This is
causing some builds to hang when running on CI.

See github#750 for an example.
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