Add Move support for semgrep #500
Conversation
chargarlic
requested review from
ihji and
brandonspark
and removed request for
a team
|
|
|
LGTM. Is the update to the |
|
yes please remove the change related to tree-sitter-csharp. |
No it is not, i cleared it up and removed those now |
There are feature and syntax differences, i can rename to move-on-sui |
chargarlic
force-pushed
the
chargarlic/add-support-move
branch
from
5701078 to
4f31e18
Compare
chargarlic
force-pushed
the
chargarlic/add-support-move
branch
from
10530cf to
a3e88b0
Compare
| const base_grammar = require('tree-sitter-move-on-sui/grammar'); | ||
|
|
||
| module.exports = grammar(base_grammar, { | ||
| name: 'move', |
There was a problem hiding this comment.
this needed to be move_on_sui, otherwise
./test-lang move-on-sui was failing with an OCaml linking error.
|
Fix here: #501 |
|
ok I've generated the code in https://github.com/semgrep/semgrep-move-on-sui |
|
now you need to add this repo as a submodule in the semgrep repo and start to add the language in semgrep-interface like the other person did for Move-on-aptos. |
Introduction of Move on Sui based on https://semgrep.dev/docs/contributing/adding-a-language . This is different from
move-on-aptosand targets a flavor of Move that is used by the Sui network.