Skip to content

Commit

Permalink
chore: fix the lint, and fail faster for lint
Browse files Browse the repository at this point in the history
  • Loading branch information
michaelfig committed May 9, 2020
1 parent 84752e2 commit 19606a6
Show file tree
Hide file tree
Showing 3 changed files with 41 additions and 5 deletions.
39 changes: 39 additions & 0 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,39 @@
name: Lint check

# run CI on pushes to master, and on all PRs (even the ones that target other
# branches)

on:
push:
branches: [master]
pull_request:

jobs:
lint:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v1
- uses: actions/setup-node@v1
with:
node-version: '12.16.1'

# FIXME: Reenable after 2020-04-01 when Github cache doesn't take forever.
#- name: cache node modules
# uses: actions/cache@v1
# with:
# path: ~/.cache/yarn
# key: ${{ runner.os }}-yarn-${{ hashFiles('yarn.lock') }}
# restore-keys: |
# ${{ runner.os }}-yarn-

# 'yarn install' must be done at the top level, to build all the
# cross-package symlinks
- name: yarn install
run: yarn install
# 'yarn build' loops over all workspaces
- name: yarn build
run: yarn build

# 'yarn lint-check' just checks the lint where desired
- name: lint check
run: yarn lint-check
3 changes: 0 additions & 3 deletions .github/workflows/test-all-packages.yml
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,3 @@ jobs:
run: cd packages/weak-store && yarn test
- name: yarn test (zoe)
run: cd packages/zoe && yarn test

- name: lint check
run: yarn lint-check
4 changes: 2 additions & 2 deletions packages/cosmic-swingset/lib/ag-solo/fake-chain.js
Original file line number Diff line number Diff line change
Expand Up @@ -42,9 +42,9 @@ export async function connectToFakeChain(basedir, GCI, role, delay, inbound) {
const vatsdir = path.join(basedir, 'vats');
const argv = [`--role=${role}`, bootAddress];
const stateDBdir = path.join(basedir, `fake-chain-${GCI}-state`);
function doOutboundBridge(dstID, obj) {
function doOutboundBridge(dstID, _obj) {
// console.error('received', dstID, obj);
return 'IBC and bridge device not used on fake-chain';
return `Bridge device (${dstID}) not implemented for fake-chain`;
}
function flushChainSends(replay) {
if (replay) {
Expand Down

0 comments on commit 19606a6

Please sign in to comment.