Skip to content
This repository was archived by the owner on May 4, 2024. It is now read-only.
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
98 commits
Select commit Hold shift + click to select a range
79d4d64
feat: add release workflow
yubing744 Jun 17, 2022
166bd9c
feat: update release script
yubing744 Jun 17, 2022
1c16bcc
feat: update release
yubing744 Jun 17, 2022
4f940bc
feat: fmt release.sh
yubing744 Jun 18, 2022
ce9ec8a
feat: add lsp.test.ts for move-analyzer
yubing744 Jun 18, 2022
9cd389b
feat: fix copyfiles error
yubing744 Jun 19, 2022
c9f13e7
feat: config test for lsp
yubing744 Jun 19, 2022
6efd3bf
feat: ok for lsp test
yubing744 Jun 20, 2022
254cd9c
feat: get symbols from table
yubing744 Jun 20, 2022
c74bfff
feat: export symbols
yubing744 Jun 21, 2022
5809291
feat: test textDocument/documentSymbol
yubing744 Jun 21, 2022
a8f6997
feat: re impl symbols
yubing744 Jun 22, 2022
3bb4356
feat: parse symbols tree
yubing744 Jun 22, 2022
b008c1a
feat: ok for on_document_symbol_request
yubing744 Jun 23, 2022
e74bceb
feat: merge from main
yubing744 Jun 23, 2022
000b9a3
feat: auto build move-analyzer and set default for serverPath
yubing744 Jun 23, 2022
f53f2cc
feat: fix ci check fail
yubing744 Jun 23, 2022
35476e9
feat: fix ci check error
yubing744 Jun 23, 2022
09ef64d
feat: fix lint error
yubing744 Jun 23, 2022
6f87de5
feat: merge from upsteam main
yubing744 Jun 24, 2022
ea8761b
fix: fix lsp test fail
yubing744 Jun 24, 2022
43fea39
feat: fmt move-analyzer code
yubing744 Jun 24, 2022
179708e
feat: fix lsp fisrt test error
yubing744 Jun 24, 2022
a479ee7
feat: update timeout for move-analyzer active
yubing744 Jun 24, 2022
bfbe898
feat: support events
yubing744 Jun 24, 2022
c863d2f
fix: fix error for LSP test
yubing744 Jun 25, 2022
945ead0
feat: merge from main
yubing744 Jun 25, 2022
71e3900
feat: fix bug for merge from master
yubing744 Jun 25, 2022
b682cd0
feat: fmt code
yubing744 Jun 25, 2022
3558e5d
Merge branch 'support-document-symbol-psl-3' into support-document-sy…
yubing744 Jun 25, 2022
b817b02
feat: update test and igonre package-lock.json for broken url test
yubing744 Jun 25, 2022
081f93e
feat: change name for ready events
yubing744 Jun 25, 2022
0be56e1
Merge branch 'support-document-symbol-psl-3' into support-document-sy…
yubing744 Jun 25, 2022
72c3cb6
feat: remove unrelate code
yubing744 Jun 26, 2022
07c9b97
feat: remove yarn.lock
yubing744 Jun 26, 2022
b006e8f
feat: ignore yarn.lock and recover configuration.ts
yubing744 Jun 26, 2022
1a19071
feat: update the comment for OnReady
yubing744 Jun 26, 2022
be3090a
Merge branch 'main' of https://github.com/move-language/move into sup…
yubing744 Jun 26, 2022
33c45fd
Merge branch 'support-document-symbol-psl-3' into support-document-sy…
yubing744 Jun 26, 2022
5ffa26e
feat: merge from main
yubing744 Jun 28, 2022
a3ab0bd
feat: remove codes
yubing744 Jun 28, 2022
b33ffce
feat: lock file
yubing744 Jun 29, 2022
023b6da
feat: add waiting code
yubing744 Jun 30, 2022
96cbf9e
feat: try lock code
yubing744 Jul 1, 2022
021c713
feat: test new way
yubing744 Jul 2, 2022
9ce1851
feat: ok for symbols
yubing744 Jul 3, 2022
b6897a4
feat: merge from main
yubing744 Jul 3, 2022
09d482f
feat: fix ci-check issue
yubing744 Jul 3, 2022
a05eeac
feat: update ci-check issues
yubing744 Jul 3, 2022
ecebbc6
feat: fix ci-check issues
yubing744 Jul 3, 2022
d623b21
Merge branch 'support-document-symbol-psl-5' into support-document-sy…
yubing744 Jul 3, 2022
00ee707
[Move analyzer] Added support for computing first symbol info synchro…
awelc Jul 11, 2022
998e630
Merge branch 'aw/move-analyzer-sync-start' of https://github.com/awel…
yubing744 Jul 13, 2022
edd291d
feat: handle symbols on start
yubing744 Jul 13, 2022
73d4c73
Merge branch 'main' into support-document-symbol-psl-6
yubing744 Jul 13, 2022
44ac909
feat: remove unuse code
yubing744 Jul 13, 2022
a1458e8
feat: format code
yubing744 Jul 13, 2022
2242869
Merge branch 'support-document-symbol-psl-6' into support-document-sy…
yubing744 Jul 13, 2022
9016aa3
feat: add test for mac and windows platform
yubing744 Jul 18, 2022
1cc7245
feat: fix bug for uri to path
yubing744 Jul 19, 2022
b8ea9ac
feat: fixbug for windows platform get_symbols stack overflow
yubing744 Jul 20, 2022
21697b0
feat: fixbug for reporting lsp diagnostics
yubing744 Jul 20, 2022
b149aee
feat: merge from main
yubing744 Jul 23, 2022
1e62ec3
feat: fixbug for windows test
yubing744 Jul 24, 2022
8d4e3d6
feat: fix lint error for language/move-analyzer/src/symbols.rs
yubing744 Jul 24, 2022
f56bb60
feat: remove config for windows stack
yubing744 Jul 24, 2022
2ad1510
feat: update pipeline for move-analyzer
yubing744 Jul 24, 2022
66d7d5a
feat: change name for test-move-analyzer-vscode-extension
yubing744 Jul 24, 2022
23c4da1
feat: remove tsfmt for windows check fail
yubing744 Jul 24, 2022
78f15cb
Merge branch 'support-document-symbol-psl-7' into support-document-sy…
yubing744 Jul 24, 2022
0660448
feat: fixbug for stack overflow and lint error
yubing744 Jul 24, 2022
33caa51
feat: fix lint error
yubing744 Jul 24, 2022
b564f72
feat: add commit for linebreak-style
yubing744 Jul 24, 2022
c4f76b9
Merge branch 'support-document-symbol-psl-8' into support-document-sy…
yubing744 Jul 24, 2022
4332a4f
feat: fix code review issue
yubing744 Jul 26, 2022
761a77b
Merge branch 'main' of https://github.com/move-language/move into sup…
yubing744 Jul 26, 2022
df1a545
feat: remove linkarg config
yubing744 Jul 26, 2022
bc250ea
feat: update cargo/config fmt
yubing744 Jul 26, 2022
4753dd8
Merge branch 'support-document-symbol-psl-9' into support-document-sy…
yubing744 Jul 26, 2022
7c55136
feat: config move-analyzer test vscode version to 1.69.2
yubing744 Jul 27, 2022
1f2dce5
feat: add comments for plugin activate
yubing744 Jul 27, 2022
2b3f95f
feat: upgrate ci-test mac system to mac-11
yubing744 Jul 27, 2022
808e8c7
feat: update style issue
yubing744 Jul 27, 2022
b40fe1e
feat: fixbug for user data dir too long for 1.69.2
yubing744 Jul 28, 2022
8e534ed
feat: fixbug for userDataDir too long
yubing744 Jul 28, 2022
feffb26
Merge branch 'main' of https://github.com/move-language/move into sup…
yubing744 Jul 28, 2022
317618b
Merge branch 'support-document-symbol-psl-9' into support-document-sy…
yubing744 Jul 28, 2022
4ec7364
feat: fixbug for vsce package missing files
yubing744 Jul 29, 2022
4695277
feat: add feature in README.md
yubing744 Jul 31, 2022
49246f6
Merge branch 'main' of https://github.com/move-language/move into sup…
yubing744 Jul 31, 2022
ceefd18
feat: add comment for startClient func in context.ts
yubing744 Jul 31, 2022
0781331
feat: update package-lock.json
yubing744 Jul 31, 2022
078e528
feat: format context.ts
yubing744 Jul 31, 2022
bfcac5e
feat: update package-lock.json for move-analyzer
yubing744 Jul 31, 2022
253e656
feat: format context.ts
yubing744 Jul 31, 2022
95c0545
Merge branch 'support-document-symbol-psl-10' into support-document-s…
yubing744 Jul 31, 2022
6d1b88f
feat: add outline feature detail descripe
yubing744 Jul 31, 2022
2af17df
feat: format README
yubing744 Aug 1, 2022
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 30 additions & 4 deletions .github/workflows/ci-pre-land.yml
Original file line number Diff line number Diff line change
Expand Up @@ -205,13 +205,19 @@ jobs:
# path: |
# /tmp/benches

build-move-analyzer-vscode-extension:
name: Build VS Code extension for move-analyzer
runs-on: ubuntu-latest
test-move-analyzer-vscode-extension:
name: Test VS Code extension for move-analyzer
strategy:
fail-fast: false
matrix:
os: [ubuntu-20.04, macos-11, windows-2022]
runs-on: ${{ matrix.os }}
needs:
- prepare
steps:
- uses: actions/checkout@v2.4.0
- name: install rust toolchain
Copy link
Collaborator

Choose a reason for hiding this comment

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

@bmwill, I was hoping that you could take a look at the changes in this file...

Copy link
Collaborator

Choose a reason for hiding this comment

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

Its not obvious to me why the rust toolchain needs to get installed to be able to build a JS package

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think I can at least try to respond to this one. The extended testing introduced in this PR involves the plugin (VSCode side) talking to the language server (which is written in Rust and presumably we need Rust tool chain to build its binary).

uses: actions-rs/toolchain@v1
- name: Use Node.js 14
uses: actions/setup-node@v2.4.0
with:
Expand All @@ -232,6 +238,26 @@ jobs:
with:
working-directory: language/move-analyzer/editors/code
run: npm run test

release-move-analyzer-vscode-extension:
name: Release VS Code extension for move-analyzer
runs-on: ubuntu-20.04
needs:
- prepare
steps:
- uses: actions/checkout@v2.4.0
- name: install rust toolchain
uses: actions-rs/toolchain@v1
- name: Use Node.js 14
uses: actions/setup-node@v2.4.0
with:
node-version: "14"
- name: Install NPM dependencies
working-directory: language/move-analyzer/editors/code
run: npm install
- name: Build the extension
working-directory: language/move-analyzer/editors/code
run: npm run pretest
- name: Package the extension
working-directory: language/move-analyzer/editors/code
run: npm run package
Expand All @@ -257,7 +283,7 @@ jobs:
gem install awesome_bot
# Don't look in git or target dirs. Don't check png, bib, tex, js, or shell files
# We allow links to be redirects, allow duplicates, and we also allow Too Many Requests (429) errors
find . -not \( -path "./.git*" -prune \) -not \( -path "./target" -prune \) -type f -not -name "*.png" -not -name "*.sh" -not -name "*.bib" -not -name "*.tex" -not -name "*.js" | while read arg; do awesome_bot --allow-redirect --allow-dupe --allow 429 --skip-save-results $arg; done
find . -not \( -path "./.git*" -prune \) -not \( -path "./target" -prune \) -type f -not -name "*.png" -not -name "*.sh" -not -name "*.bib" -not -name "*.tex" -not -name "*.js" -not -name "package-lock.json" | while read arg; do awesome_bot --allow-redirect --allow-dupe --allow 429 --skip-save-results $arg; done

build-move-cli-docker-image:
name: Build Docker image for the Move CLI
Expand Down
1 change: 1 addition & 0 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions language/move-analyzer/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ im = "15.1.0"
lsp-server = "0.5.1"
lsp-types = "0.90.1"
petgraph = "0.5.1"
serde = { version = "1.0", features = ["derive"] }
serde_json = "1.0.64"
tempfile = "3.2.0"
url = "2.2.2"
Expand Down
15 changes: 10 additions & 5 deletions language/move-analyzer/editors/code/.eslintrc.json
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@
"function-call-argument-newline": ["warn", "consistent"],
"function-paren-newline": ["warn", "consistent"],
"key-spacing": "warn",
"linebreak-style": "warn",
"linebreak-style": "off", // We use different linebreak styles. ref https://eslint.org/docs/latest/rules/linebreak-style
"max-len": [
"warn",
{
Expand Down Expand Up @@ -139,7 +139,6 @@
// * All @typescript-eslint rules: https://github.com/typescript-eslint/typescript-eslint/tree/v4.31.1/packages/eslint-plugin/docs/rules
// * The rules included in @typescript-eslint/recommended: https://github.com/typescript-eslint/typescript-eslint/blob/v4.31.1/packages/eslint-plugin/src/configs/recommended.ts
// * The rules included in @typescript-eslint/recommended-requiring-type-checking: https://github.com/typescript-eslint/typescript-eslint/blob/v4.31.1/packages/eslint-plugin/src/configs/recommended-requiring-type-checking.ts
"@typescript-eslint/array-type": ["warn", { "default": "generic" }],
"@typescript-eslint/ban-tslint-comment": "warn",
"@typescript-eslint/brace-style": "warn",
"@typescript-eslint/comma-dangle": [
Expand All @@ -158,7 +157,7 @@
"@typescript-eslint/keyword-spacing": "warn",
"@typescript-eslint/lines-between-class-members": "warn",
"@typescript-eslint/member-delimiter-style": "warn",
"@typescript-eslint/naming-convention": "warn",
"@typescript-eslint/naming-convention": "off",
"@typescript-eslint/no-base-to-string": "warn",
"@typescript-eslint/no-confusing-non-null-assertion": "warn",
"@typescript-eslint/no-duplicate-imports": "warn",
Expand Down Expand Up @@ -206,10 +205,16 @@
],
"@typescript-eslint/strict-boolean-expressions": "warn",
"@typescript-eslint/switch-exhaustiveness-check": "warn",
"@typescript-eslint/type-annotation-spacing": "warn",
"@typescript-eslint/unified-signatures": "warn",

// The following are eslint-plugin-tsdoc rules:
"tsdoc/syntax": "warn"
"tsdoc/syntax": "warn",

"@typescript-eslint/no-unsafe-return": "off",
"@typescript-eslint/type-annotation-spacing": "off",
"@typescript-eslint/no-explicit-any": "off",
"@typescript-eslint/no-unsafe-assignment": "off",
"@typescript-eslint/restrict-template-expressions": "off",
"@typescript-eslint/array-type": "off"
}
}
2 changes: 2 additions & 0 deletions language/move-analyzer/editors/code/.gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,5 @@ out
.vscode-test
# VS Code extension package (VSIX) archive
*.vsix
# Yarn cache directory
yarn.lock
2 changes: 1 addition & 1 deletion language/move-analyzer/editors/code/.vscodeignore
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

**/*
!images/move.png
!out/src/*
!out/src/**/*
!node_modules/**/*
!language-configuration.json
!move.tmLanguage.json
Expand Down
1 change: 1 addition & 0 deletions language/move-analyzer/editors/code/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -86,3 +86,4 @@ Move source file (a file with a `.move` file extension) and:
- go to type definition
- go to references
- type on hover
- outline view showing symbol tree for Move source files
Loading