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

Add support for generic methods under interop #1103

Merged
merged 6 commits into from Mar 9, 2022
Merged

Add support for generic methods under interop #1103

merged 6 commits into from Mar 9, 2022

Conversation

ghost
Copy link

@ghost ghost commented Mar 8, 2022

add support for generic methods

Copy link
Collaborator

@lahma lahma left a comment

Choose a reason for hiding this comment

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

Mostly stylistic issue, tabs seems to running wild again. Also could you add description to PR what is to be expected?

Jint.Tests/Runtime/GenericMethodTests.cs Outdated Show resolved Hide resolved
Jint.Tests/Runtime/GenericMethodTests.cs Outdated Show resolved Hide resolved
Jint.Tests/Runtime/GenericMethodTests.cs Outdated Show resolved Hide resolved
Jint.Tests/Runtime/GenericMethodTests.cs Outdated Show resolved Hide resolved
Jint.Tests/Runtime/GenericMethodTests.cs Outdated Show resolved Hide resolved
Jint/Runtime/TypeConverter.cs Outdated Show resolved Hide resolved
Jint/Runtime/TypeConverter.cs Outdated Show resolved Hide resolved
Jint/Runtime/TypeConverter.cs Outdated Show resolved Hide resolved
Jint.Tests/Runtime/GenericMethodTests.cs Outdated Show resolved Hide resolved
Jint/Runtime/Interop/MethodInfoFunctionInstance.cs Outdated Show resolved Hide resolved
@lahma
Copy link
Collaborator

lahma commented Mar 8, 2022

If you rebase against main you should get .editorconfig that should guide your editor to use spaces over tabs, maybe that helps.

@ghost
Copy link
Author

ghost commented Mar 8, 2022

no problem for switching tabs to spaces - each time I do it (use tabs instead of spaces) - it is accidental. The convention just differs from the other projects I work in - so my editor defaults to tabs - I've grabbed the editor config - will try updating those individual spots you mentioned - see if the editor does the right thing. In case it is of interest I'm using Visual Studio Preview/2022 on a mac.

Another note of possible interest is I can't actually run the entire test suite in the JINT solution (so I'm reliant on the CI/CD to perform the test run on the entire test suite for me). Eventually the tests just start failing or timing out and I end up with a zombie dotnet process. I'm not too sure if there is a particular test(s) that causes this - but in case this information is of interest.

I think I've correctly resolved all the issues identified - let me know if I missed anything - thanks!

Lastly - let me know if you want more details in the description of this PR. Our project is using typescript and so we want to be able to use C# generic methods from the javascript we execute. I have some test code which shows a few different applications.

@ghost ghost requested a review from lahma March 8, 2022 19:41
Tim Cassidy and others added 6 commits March 9, 2022 08:47
…dows rune had a timeout:

Skipped language\statements\class(sourceFile: language/statements/class/dstr-gen-meth-static-obj-ptrn-id-get-value-err.js) [1 ms]
vstest.console process failed to connect to testhost process after 90 seconds. This may occur due to machine slowness, please set environment variable VSTEST_CONNECTION_TIMEOUT to increase timeout.
Copy link
Collaborator

@lahma lahma left a comment

Choose a reason for hiding this comment

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

I did small cleanup, there were still some tabs left and made new TypeConverter method internal.

@lahma lahma changed the title add support for generic methods Add support for generic methods under interop Mar 9, 2022
@lahma lahma enabled auto-merge (squash) March 9, 2022 06:53
@lahma lahma merged commit d1e69ae into sebastienros:main Mar 9, 2022
@lahma lahma added the interop label Mar 9, 2022
@ghost ghost deleted the issues/tpc/genericMethodSupport branch March 9, 2022 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant