Mergin main into small dfg fix #1650
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Do not depend on JavaImportResolver in Neo4j
* Fixed Neo4J import of `ImportDeclaration` The `ImportDeclaration` was trying to use the `Name` class without a converter. * Need to ignore * Adding SimpleNameConverter * Added small test for name converter
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Using Kotlin 1.9 for cpg-console It seems that the kotlin shell is not (yet) compatbile with Kotlin 2.0. However, it seems that we can just use the scripting host from Kotlin 1.9 and it seems to work. Fixes #1591 Related to Kotlin/kotlin-interactive-shell#131 * Added an integration test for the console * Adding cpg-console to test coverage
* Update README.md * add qasm and qiskit (#1601) * Adapting language support category * Additional software remark changed * Update README.md * Change developing to incubating --------- Co-authored-by: Maximilian Kaul <[email protected]> Co-authored-by: Christian Banse <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Added `hasArguments` to `ArgumentHolder` This adds a small utility function to `ArgumentHolder`, so we can check if an expression is part of their "arguments".
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Fixed crash in CXXExtraPass::convertOperators This is more a band-aid, than an actually "healing". The issue is that we often use the `language` to get a translation context in passes. But (sometimes) the language does not have it, since it is created using `registerLanguage` even before a translation context exists. In an attempted fix, I am now using the translation context of a different node, but we should really think about how to deal with that in the future. Fixes #1617
* Added `OperatorDeclaration` This PR adds support for an operator declaration. They are not resolved yet. * Python operators (#1625) --------- Co-authored-by: Maximilian Kaul <[email protected]>
We also take the first path and then break, so that it won't get overidden.
* Populating `codeOf` of `ASTModule` This error message was annoying me, so here is a fix. I am not sure if it is easy to find the "location" of the module. The region should be able to be computed out of the lengt of `fileContents` and its actual content. I actually thought we had a function like that somewhere @konradweiss? * make sonarcube happy --------- Co-authored-by: Maximilian Kaul <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
|
Closed a38b853 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a playground pr. Feel free to solve the issues and merge the branches while I'm on vacation :)
This breaks some tests.