Skip to content

Commit

Permalink
Run tests and linter in github workflows (#3)
Browse files Browse the repository at this point in the history
  • Loading branch information
mrkvon authored Sep 24, 2023
1 parent e6cfeda commit f674352
Show file tree
Hide file tree
Showing 3 changed files with 53 additions and 0 deletions.
29 changes: 29 additions & 0 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
name: Lint

on: push

jobs:
lint:
name: Lint
runs-on: ubuntu-latest

steps:
- name: Checkout
uses: actions/checkout@v3

- name: Setup Node
uses: actions/setup-node@v3
with:
node-version: 16

- name: Install NPM packages
run: yarn install --frozen-lockfile

- name: Run linters
uses: wearerequired/lint-action@v2
with:
eslint: true
eslint_args: --max-warnings 0
eslint_extensions: js,ts,tsx
prettier: true
continue_on_error: false
22 changes: 22 additions & 0 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
name: Run integration tests

on: push

jobs:
test:
runs-on: ubuntu-latest

steps:
- name: Setup Node
uses: actions/setup-node@v3
with:
node-version: 16

- name: Checkout
uses: actions/checkout@v3

- name: Install dependencies
run: yarn install --frozen-lockfile

- name: Run tests
run: yarn test
2 changes: 2 additions & 0 deletions src/controllers/integration.ts
Original file line number Diff line number Diff line change
Expand Up @@ -159,6 +159,8 @@ const getWebhookChannel = async (

const dru = new URL(descriptionResourceUri as string, url).toString()
// we want to include looking into this resource, too
// TODO use this variable below
// eslint-disable-next-line @typescript-eslint/no-unused-vars
const descriptionResource = await (await fetch(dru)).text()

return webhookChannelTriple.subject.id
Expand Down

0 comments on commit f674352

Please sign in to comment.