Skip to content

Commit

Permalink
ci: define env variables directly in workflow
Browse files Browse the repository at this point in the history
  • Loading branch information
Slartibartfass2 authored and Merseleo committed Dec 10, 2024
1 parent 29cf044 commit 26953a2
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 17 deletions.
13 changes: 1 addition & 12 deletions .github/workflows/code_quality_checks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ on:

env:
node_version: ${{ vars.NODE_VERSION }}
PUBLIC_API_BASE_URL: http://localhost:8080

jobs:
linting:
Expand Down Expand Up @@ -45,9 +46,6 @@ jobs:
- name: Install dependencies
run: npm install

- name: Create .env file from .env.example
run: cat .env.example > .env

- name: Run unit tests
run: npm run test:unit

Expand Down Expand Up @@ -75,9 +73,6 @@ jobs:
- name: Install dependencies
run: npm install

- name: Create .env file from .env.example
run: cat .env.example > .env

- name: Run integration tests
run: npm run test:integration

Expand Down Expand Up @@ -106,9 +101,6 @@ jobs:
- name: Install dependencies
run: npm install

- name: Create .env file from .env.example
run: cat .env.example > .env

- name: Install Playwright
run: npx playwright install --with-deps

Expand Down Expand Up @@ -190,8 +182,5 @@ jobs:
- name: Install dependencies
run: npm install

- name: Create .env file from .env.example
run: cat .env.example > .env

- name: Run all checks
run: npm run check
7 changes: 4 additions & 3 deletions .github/workflows/lighthouse.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,10 @@ on:
types:
- review_requested

env:
node_version: ${{ vars.NODE_VERSION }}
PUBLIC_API_BASE_URL: http://localhost:8080

jobs:
lighthouse:
name: Lighthouse Auditing
Expand All @@ -21,9 +25,6 @@ jobs:
- name: Install dependencies
run: npm install

- name: Create .env file from .env.example
run: cat .env.example > .env

- name: Build application
run: npm run build

Expand Down
2 changes: 1 addition & 1 deletion src/lib/backend-api.ts
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ export interface IAuthorController {
}

export interface IBackendController {
signIn(username: string, password: string): Promise<User>;
signIn(email: string, password: string): Promise<User>;
signOut(): Promise<void>;
requestEmailForgottenPassword(email: string): Promise<void>;

Expand Down
2 changes: 1 addition & 1 deletion src/lib/controller/backend-controller.ts
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ export class BackendController implements IBackendController {
}

// eslint-disable-next-line @typescript-eslint/no-unused-vars
async signIn(username: string, password: string): Promise<User> {
async signIn(email: string, password: string): Promise<User> {
throw new Error("Method not implemented.");
}

Expand Down

0 comments on commit 26953a2

Please sign in to comment.