Skip to content

Commit

Permalink
Merge pull request #46 from ajhenry/ajhenry/fix-env-build
Browse files Browse the repository at this point in the history
Remove unused setup test and fix env stubbing in CI environments
  • Loading branch information
ajhenry authored Feb 22, 2024
2 parents ced891b + f619dbd commit cdee50d
Show file tree
Hide file tree
Showing 9 changed files with 203 additions and 280 deletions.
3 changes: 0 additions & 3 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,5 @@ jobs:
- name: Install Dependencies
run: npm install

- name: Setup CI Environment
run: mv .env.example .env

- name: Run Build
run: NEXT_TELEMETRY_DISABLED=1 npm run build
3 changes: 0 additions & 3 deletions .github/workflows/docker-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,5 @@ jobs:
- name: Checkout code
uses: actions/checkout@v4

- name: Setup CI Environment
run: mv .env.example .env

- name: Build Docker image
run: docker build -t internal-contribution-forks .
3 changes: 0 additions & 3 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,5 @@ jobs:
- name: Install Dependencies
run: npm install

- name: Setup CI Environment
run: mv .env.example .env

- name: Run Lint
run: npm run lint
3 changes: 0 additions & 3 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,5 @@ jobs:
- name: Install Dependencies
run: npm install

- name: Setup CI Environment
run: mv .env.example .env

- name: Run Jest Tests
run: npm test
1 change: 1 addition & 0 deletions env.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -49,4 +49,5 @@ export const env = createEnv({
PUBLIC_ORG: process.env.PUBLIC_ORG,
PRIVATE_ORG: process.env.PRIVATE_ORG,
},
skipValidation: process.env.SKIP_ENV_VALIDATIONS === 'true',
})
1 change: 0 additions & 1 deletion jest.config.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
/** @type {import('ts-jest/dist/types').InitialOptionsTsJest} */
module.exports = {
preset: 'ts-jest',
setupFilesAfterEnv: ['./test/setupJest.ts'],
testRegex: '(/__tests__/.*|\\.(test|spec))\\.[tj]sx?$',
testEnvironment: 'node',
moduleDirectories: ['node_modules', 'src'],
Expand Down
Loading

0 comments on commit cdee50d

Please sign in to comment.