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

merge vscode-python upstream 2024.14.1 #4670

Open
wants to merge 22 commits into
base: main
Choose a base branch
from

Conversation

isabelizimm
Copy link
Contributor

@isabelizimm isabelizimm commented Sep 12, 2024

This applies changes from v2024.12.0, v2024.12.1, v2024.12.2, v2024.12.3, v2024.14.0, v2024.14.1.

  • The majority of work for this merge is around the Native REPL, which we don't use at all.
  • Move from to typescript 4.5.5 to ~5.2
  • The Rust env finder is the main locator upstream now. We currently don't use it, but may want to take steps for that change sooner rather than later. All things will work normally still as the builtin locator is still supported.

// is the problem. We still need to make it thenable though. See this issue: https://github.com/florinn/typemoq/issues/67
const result = TypeMoq.Mock.ofType<T>(targetCtor, behavior, shouldOverrideTarget, ...targetCtorArgs);
// --- Start Positron ---
// result.setup((x: any) => x.then).returns(() => undefined);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this line caused every single usage of this function to fail with TypeError: Cannot convert a Symbol value to a string-- I'm not sure why since it seems to be a community vetted way to mock things resolved as promises and works upstream 😩

});
// TODO @isabelizimm: HACK HACK HACK
// Replace the original `asRelativePath` method with the mock object
vscode.workspace.asRelativePath = mockAsRelativePath;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was returning TypeError: Descriptor for property workspace is non-configurable and non-writable but if you stub the property asRelativePath instead of workspace, I hit TypeError: Cannot stub non-existent property asRelativePath instead. My workaround is to mock and then patch this

@isabelizimm isabelizimm marked this pull request as ready for review October 1, 2024 02:07
@isabelizimm
Copy link
Contributor Author

isabelizimm commented Oct 1, 2024

I played around with things very briefly, but if others can kick the tires, that would be much appreciated!

(still todo: looks like the Tests on Linux is failing for Buffer Arrays, maybe needs an updated version?)

@isabelizimm isabelizimm changed the title [wip] merge vscode-python upstream 2024.14.1 merge vscode-python upstream 2024.14.1 Oct 1, 2024
@@ -254,6 +254,13 @@ jobs:
- name: Install test requirements
run: python -m pip install --upgrade -r build/test-requirements.txt

- name: Rust Tool Chain setup
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't think this should affect our build, but noting in case we should comment it out until we use the python-environment-tools Python locator

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