mirror of
https://github.com/coder/coder.git
synced 2026-09-22 05:05:20 +08:00
refactor: Move package.json and other front-end collateral into 'site' (#128)
This refactors the front-end collateral to all live within `site` - so no `package.json` at the root. The reason we had this initially is that the jest test run and NextJS actually require having _two_ different `tsconfig`s - Next needs `jsx:"preserve"`, while jest needs `jsx:"react"` - we were using `tsconfig`s at different levels at the hierarchy to manage this. I changed this behavior to still use two different `tsconfig.json`s, which is mandatory - but just side-by-side in `site`. Once that's fixed, it was easy to move everything into `site` Follow up from: https://github.com/coder/coder/pull/118#discussion_r796244577
This commit is contained in:
@@ -61,9 +61,11 @@ jobs:
|
||||
|
||||
- name: Install node_modules
|
||||
run: yarn install
|
||||
working-directory: site
|
||||
|
||||
- name: "yarn lint"
|
||||
run: yarn lint
|
||||
working-directory: site
|
||||
|
||||
gen:
|
||||
name: "style/gen"
|
||||
@@ -113,6 +115,7 @@ jobs:
|
||||
|
||||
- name: Install node_modules
|
||||
run: yarn install
|
||||
working-directory: site
|
||||
|
||||
- name: "make ${{ matrix.style }}"
|
||||
run: "make --output-sync -j ${{ matrix.style }}"
|
||||
@@ -194,15 +197,18 @@ jobs:
|
||||
node-version: "14"
|
||||
|
||||
- run: yarn install
|
||||
working-directory: site
|
||||
|
||||
- run: yarn build
|
||||
working-directory: site
|
||||
|
||||
- run: yarn test:coverage
|
||||
working-directory: site
|
||||
|
||||
- uses: codecov/codecov-action@v2
|
||||
if: github.actor != 'dependabot[bot]'
|
||||
with:
|
||||
token: ${{ secrets.CODECOV_TOKEN }}
|
||||
files: ./coverage/lcov.info
|
||||
files: ./site/coverage/lcov.info
|
||||
flags: unittest-js
|
||||
fail_ci_if_error: true
|
||||
|
||||
Reference in New Issue
Block a user