Skip to content

Commit

Permalink
Merge pull request #14 from zowe/fix/theia-vnext-build
Browse files Browse the repository at this point in the history
Fix 429 rate limit error in weekly Theia build
  • Loading branch information
awharn authored Aug 20, 2024
2 parents c945814 + 21abb2f commit c88ce39
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 7 deletions.
5 changes: 2 additions & 3 deletions .github/workflows/docker-reusable.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -72,12 +72,11 @@ jobs:
id: build
uses: docker/build-push-action@v5
with:
build-args: |
GITHUB_TOKEN=${{ secrets.GITHUB_TOKEN }}
${{ inputs.build-args }}
build-args: ${{ inputs.build-args }}
context: ${{ inputs.path }}
push: ${{ inputs.push }}
platforms: ${{ inputs.qemu && 'linux/amd64,linux/arm64' || 'linux/amd64' }}
secrets: github_token=${{ secrets.GITHUB_TOKEN }}
tags: ${{ inputs.tags || env.DEFAULT_TAGS }}

- name: Image Digest
Expand Down
3 changes: 1 addition & 2 deletions ze/theia-slim/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -13,11 +13,10 @@ WORKDIR /home/theia
ADD buildPackageJson.js ./buildPackageJson.js
ARG THEIA_VERSION=latest
RUN echo "$(node buildPackageJson.js ${THEIA_VERSION})" > package.json
ARG GITHUB_TOKEN
RUN yarn global add node-gyp && \
yarn --pure-lockfile && \
NODE_OPTIONS="--max_old_space_size=4096" yarn theia build && \
yarn theia download:plugins && \
yarn theia download:plugins --rate-limit=15 --parallel=false && \
yarn --production && \
yarn autoclean --init && \
echo *.ts >> .yarnclean && \
Expand Down
2 changes: 0 additions & 2 deletions ze/theia-slim/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,6 @@ Based on https://github.com/theia-ide/theia-apps/tree/master/theia-docker

To build the container, run `docker build .`

**Note:** You may need to provide a GITHUB_TOKEN as a build argument with `repo` access

The following runs Theia IDE on http://localhost:3000 with the current directory as a workspace. The option of `--init` is added to fix the [defunct process problem](https://github.com/theia-ide/theia-apps/issues/195).

```bash
Expand Down

0 comments on commit c88ce39

Please sign in to comment.