Skip to content

Commit

Permalink
Merge pull request #17 from pportella23/lint-quality-action
Browse files Browse the repository at this point in the history
adds `lint:eslint:check` + `eslint` job in `Linting` GitHub Action
  • Loading branch information
pportella23 authored Nov 16, 2024
2 parents f5e0b12 + 4c8d159 commit 908ab73
Show file tree
Hide file tree
Showing 7 changed files with 4,479 additions and 731 deletions.
8 changes: 8 additions & 0 deletions .eslintrc.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
{
"extends": [
"eslint:recommended",
"plugin:jest/recommended",
"next/core-web-vitals",
"prettier"
]
}
13 changes: 13 additions & 0 deletions .github/workflows/linting.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,3 +16,16 @@ jobs:
- run: npm ci

- run: npm run lint:prettier:check
eslint:
name: Eslint
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4

- uses: actions/setup-node@v4
with:
node-version: "lts/hydrogen"

- run: npm ci

- run: npm run lint:eslint:check
4 changes: 3 additions & 1 deletion infra/database.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,11 +28,13 @@ async function getNewClient() {
return client;
}

export default {
const database = {
query,
getNewClient,
};

export default database;

function getSSLValues() {
if (process.env.POSTGRES_CA) {
return {
Expand Down
1 change: 1 addition & 0 deletions infra/migrations/1712581422251_test-migration.js
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
/* eslint-disable no-unused-vars */
/* eslint-disable camelcase */

exports.shorthands = undefined;
Expand Down
Loading

0 comments on commit 908ab73

Please sign in to comment.