Skip to content

Commit 58bdfaa

Browse files
authored
chore: remove unused go-repl worker (#370)
* chore: remove go-repl worker * chore: disable obsolete lint rules * chore: remove unused 'lib/gowasm' package * chore: add readme for gowasm
1 parent 71e1d26 commit 58bdfaa

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

67 files changed

+7
-3199
lines changed

.golangci.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,3 @@ issues:
55
- pkg/worker
66
- cmd/webworker
77
- web
8-
exclude-rules:
9-
- path: 'internal/gowasm/(.+)\.go'
10-
linters:
11-
- typecheck

HACKING.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,6 @@ WASM Go programs are located in [`cmd/wasm`](./cmd/wasm/) directory.
5151
| Package | Description |
5252
| ---------- | --------------------------------------------------------------------------------------------------------------------------- |
5353
| `analyzer` | Very simple Go syntax checker. |
54-
| `go-repl` | Deprecated tool with [Yaegi](https://github.com/traefik/yaegi) Go interpreter. Used to run Go programs without compilation. |
5554

5655
### Tools & Scripts
5756

build.mk

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -70,12 +70,8 @@ wasm_exec.js:
7070
analyzer.wasm:
7171
$(call build_wasm_worker,analyzer)
7272

73-
.PHONY:go-repl.wasm
74-
go-repl.wasm:
75-
$(call build_wasm_worker,go-repl)
76-
7773
.PHONY:wasm
78-
wasm: wasm_exec.js analyzer.wasm go-repl.wasm
74+
wasm: wasm_exec.js analyzer.wasm
7975

8076
.PHONY: build
8177
build: check-go check-yarn clean preinstall gen collect-meta build-server wasm pkg-index build-ui

build/Dockerfile

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -43,11 +43,6 @@ RUN echo "Building server with version $APP_VERSION" && \
4343
-buildvcs=false \
4444
-ldflags "-s -w" \
4545
-trimpath \
46-
-o ./go-repl@$WASM_API_VER.wasm ./cmd/wasm/go-repl && \
47-
GOOS=js GOARCH=wasm go build \
48-
-buildvcs=false \
49-
-ldflags "-s -w" \
50-
-trimpath \
5146
-o ./analyzer@$WASM_API_VER.wasm ./cmd/wasm/analyzer && \
5247
cp $(go env GOROOT)/misc/wasm/wasm_exec.js ./wasm_exec@v2.js && \
5348
cp $(go env GOROOT)/misc/wasm/wasm_exec.js ./wasm_exec.js && \

build/release.dockerfile

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -26,11 +26,6 @@ COPY go.sum .
2626
## wasm_exec.js files are left for backwards compatibility for old clients.
2727
RUN echo "Building server with version $APP_VERSION" && \
2828
go build -o server -ldflags="-X 'main.Version=$APP_VERSION'" ./cmd/playground && \
29-
GOOS=js GOARCH=wasm go build \
30-
-buildvcs=false \
31-
-ldflags "-s -w" \
32-
-trimpath \
33-
-o ./go-repl@$WASM_API_VER.wasm ./cmd/wasm/go-repl && \
3429
GOOS=js GOARCH=wasm go build \
3530
-buildvcs=false \
3631
-ldflags "-s -w" \

cmd/wasm/go-repl/main.go

Lines changed: 0 additions & 37 deletions
This file was deleted.

internal/gorepl/errors.go

Lines changed: 0 additions & 27 deletions
This file was deleted.

internal/gorepl/handler_js.go

Lines changed: 0 additions & 148 deletions
This file was deleted.

0 commit comments

Comments
 (0)