|
|
|
|
name: CI
|
|
|
|
|
|
|
|
|
|
on:
|
|
|
|
|
pull_request:
|
|
|
|
|
push:
|
|
|
|
|
branches:
|
|
|
|
|
- 'staging'
|
|
|
|
|
- 'master'
|
|
|
|
|
|
|
|
|
|
jobs:
|
|
|
|
|
formatting:
|
|
|
|
|
runs-on: ubuntu-latest
|
|
|
|
|
steps:
|
|
|
|
|
- uses: actions/checkout@v2
|
|
|
|
|
- uses: dprint/check@v2.0
|
|
|
|
|
|
|
|
|
|
clippy:
|
|
|
|
|
runs-on: ubuntu-latest
|
|
|
|
|
steps:
|
|
|
|
|
- uses: actions/checkout@v2
|
|
|
|
|
- name: Install Rust
|
|
|
|
|
uses: actions-rs/toolchain@v1
|
|
|
|
|
with:
|
|
|
|
|
profile: minimal
|
|
|
|
|
override: true
|
|
|
|
|
toolchain: stable
|
|
|
|
|
components: clippy
|
|
|
|
|
- uses: Swatinem/rust-cache@v1.3.0
|
|
|
|
|
- run: cargo clippy --workspace --all-targets -- -D warnings
|
|
|
|
|
|
|
|
|
|
check_frontend:
|
|
|
|
|
defaults:
|
|
|
|
|
run:
|
|
|
|
|
working-directory: frontend
|
|
|
|
|
runs-on: ubuntu-latest
|
|
|
|
|
steps:
|
|
|
|
|
- uses: actions/checkout@v2
|
|
|
|
|
- uses: actions/setup-node@v2
|
|
|
|
|
with:
|
|
|
|
|
cache: 'yarn'
|
|
|
|
|
cache-dependency-path: frontend/yarn.lock
|
|
|
|
|
- run: yarn install
|
|
|
|
|
- run: yarn run eslint
|
|
|
|
|
- run: yarn run tsc
|
|
|
|
|
|
|
|
|
|
build_and_test_frontend:
|
|
|
|
|
strategy:
|
|
|
|
|
matrix:
|
|
|
|
|
app: [ maker, taker ]
|
|
|
|
|
defaults:
|
|
|
|
|
run:
|
|
|
|
|
working-directory: frontend
|
|
|
|
|
runs-on: ubuntu-latest
|
|
|
|
|
steps:
|
|
|
|
|
- uses: actions/checkout@v2
|
|
|
|
|
- uses: actions/setup-node@v2
|
|
|
|
|
with:
|
|
|
|
|
cache: 'yarn'
|
|
|
|
|
cache-dependency-path: frontend/yarn.lock
|
|
|
|
|
- run: yarn install
|
|
|
|
|
- run: APP=${{ matrix.app }} yarn test
|
|
|
|
|
- run: APP=${{ matrix.app }} yarn build
|
|
|
|
|
|
|
|
|
|
test_daemons:
|
|
|
|
|
strategy:
|
|
|
|
|
matrix:
|
|
|
|
|
os: [ ubuntu-latest, macos-latest, windows-latest ]
|
|
|
|
|
runs-on: ${{ matrix.os }}
|
|
|
|
|
steps:
|
|
|
|
|
- uses: actions/checkout@v2
|
|
|
|
|
- name: Setup rust toolchain
|
|
|
|
|
run: rustup show
|
|
|
|
|
- uses: Swatinem/rust-cache@v1.3.0
|
|
|
|
|
- run: cargo build --bins --tests
|
|
|
|
|
- run: cargo test --workspace
|
|
|
|
|
- name: Smoke test ${{ matrix.os }} binary
|
|
|
|
|
shell: bash
|
|
|
|
|
run: |
|
|
|
|
|
target/debug/maker --data-dir=/tmp/maker --generate-seed testnet &
|
|
|
|
|
sleep 10s # Wait for maker to start
|
Remove hardcoded absolute URLs from the frontend source code
Achieving this is rather tricky due to the nature of our multi-page
project. We need to solve several problems:
1. We want a single npm project that produces two independent bundles.
2. We want our paths to be relative to the serving URL, that is `localhost:3000`
in development and in production, whereever the backend is hosted.
3. We have independent backends, hence requiring different `server.proxy`
configurations.
We solve (1) by using vite (already prior to this commit).
To solve (2), we simply remove all absolute URLs from the code and replace
them with absolute paths which will be relative to the serving host.
This creates a problem: Prior to this patch, we only have one devServer running
that would serve both frontends under a different sub-directory (/maker and /taker).
Even though this worked, it was impossible to create a proxy configuration that would:
- Forward API requests from `/maker` to `localhost:8001`
- Forward API requests from `/taker` to `localhost:8000`
Because in both cases, the API requests would simply start with `/api`, making
them indistinguishable from each other.
To solve this problem, we needed to serve each frontend separately. Doing so
would allow us to have dedicated proxy server configurations and forward the
requests to `/api` to the correct backend.
Unfortunately, the intuitive approach of solving this (have a `maker.html` and
`taker.html` file) does not work. With React being a client-side routing framework,
full page-reloads would be broken with this approach because they would be looking
for an `index.html` file which doesn't exist.
To work around this issue, our final solution is:
1. Use a dynamic ID to reference the desired app from within the `index.html`: `__app__`
2. Use a vite plugin to resolve this ID to the file in question: `maker.tsx` or `taker.tsx`
Fixes #6.
3 years ago
|
|
|
|
|
|
|
|
target/debug/taker --data-dir=/tmp/taker --generate-seed testnet &
|
|
|
|
|
sleep 10s # Wait for taker to start
|
Remove hardcoded absolute URLs from the frontend source code
Achieving this is rather tricky due to the nature of our multi-page
project. We need to solve several problems:
1. We want a single npm project that produces two independent bundles.
2. We want our paths to be relative to the serving URL, that is `localhost:3000`
in development and in production, whereever the backend is hosted.
3. We have independent backends, hence requiring different `server.proxy`
configurations.
We solve (1) by using vite (already prior to this commit).
To solve (2), we simply remove all absolute URLs from the code and replace
them with absolute paths which will be relative to the serving host.
This creates a problem: Prior to this patch, we only have one devServer running
that would serve both frontends under a different sub-directory (/maker and /taker).
Even though this worked, it was impossible to create a proxy configuration that would:
- Forward API requests from `/maker` to `localhost:8001`
- Forward API requests from `/taker` to `localhost:8000`
Because in both cases, the API requests would simply start with `/api`, making
them indistinguishable from each other.
To solve this problem, we needed to serve each frontend separately. Doing so
would allow us to have dedicated proxy server configurations and forward the
requests to `/api` to the correct backend.
Unfortunately, the intuitive approach of solving this (have a `maker.html` and
`taker.html` file) does not work. With React being a client-side routing framework,
full page-reloads would be broken with this approach because they would be looking
for an `index.html` file which doesn't exist.
To work around this issue, our final solution is:
1. Use a dynamic ID to reference the desired app from within the `index.html`: `__app__`
2. Use a vite plugin to resolve this ID to the file in question: `maker.tsx` or `taker.tsx`
Fixes #6.
3 years ago
|
|
|
|
|
|
|
|
curl --fail http://localhost:8000/api/alive
|
|
|
|
|
curl --fail http://localhost:8001/api/alive
|
|
|
|
|
- name: Upload binaries
|
|
|
|
|
uses: actions/upload-artifact@v2
|
|
|
|
|
with:
|
|
|
|
|
name: maker-and-taker-binaries-${{ matrix.os }}
|
|
|
|
|
path: |
|
|
|
|
|
target/debug/maker
|
|
|
|
|
target/debug/taker
|
|
|
|
|
|
|
|
|
|
daemons_arm_build:
|
|
|
|
|
runs-on: ubuntu-latest
|
|
|
|
|
steps:
|
|
|
|
|
- uses: actions/checkout@v2
|
|
|
|
|
- name: Setup rust toolchain
|
|
|
|
|
run: rustup show
|
|
|
|
|
- uses: Swatinem/rust-cache@v1.3.0
|
|
|
|
|
- name: Install compiler for armhf arch
|
|
|
|
|
run: |
|
|
|
|
|
sudo apt-get update
|
|
|
|
|
sudo apt-get install gcc-arm-linux-gnueabihf
|
|
|
|
|
- run: cargo build --target=armv7-unknown-linux-gnueabihf --bins
|
|
|
|
|
- name: Upload binaries
|
|
|
|
|
uses: actions/upload-artifact@v2
|
|
|
|
|
with:
|
|
|
|
|
name: maker-and-taker-binaries-armv7
|
|
|
|
|
path: |
|
|
|
|
|
target/armv7-unknown-linux-gnueabihf/debug/maker
|
|
|
|
|
target/armv7-unknown-linux-gnueabihf/debug/taker
|
