Skip to content

Commit f6faaf2

Browse files
author
Lukas Holzer
authored
ci: store npm logs on failure (#5678)
* ci: store npm logs on failure * chore: regenerate lock file version for integration test site * chore: update eleventy-site dependency * chore: change to npm 7 compatible version * chore: update lock file of site as well * chore: update * chore: force transitive dependencies to use the same version * chore: test * chore: last update * chore: update * chore: remove site from linting
1 parent 0e3c917 commit f6faaf2

File tree

8 files changed

+4131
-5895
lines changed

8 files changed

+4131
-5895
lines changed

.github/workflows/integration-tests.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,14 @@ jobs:
104104
echo "node=node_${node/.*.*/}" >> $env:GITHUB_OUTPUT
105105
shell: bash
106106
if: '${{ !steps.release-check.outputs.IS_RELEASE }}'
107+
- name: Store npm error artefacts
108+
uses: actions/upload-artifact@v3
109+
if: always()
110+
with:
111+
name: npm-logs
112+
path: |
113+
/home/runner/.npm/_logs/**/*
114+
107115
- uses: codecov/codecov-action@v3
108116
continue-on-error: true
109117
with:

.github/workflows/legacy-tests.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -108,6 +108,13 @@ jobs:
108108
echo "node=node_${node/.*.*/}" >> $env:GITHUB_OUTPUT
109109
shell: bash
110110
if: '${{ !steps.release-check.outputs.IS_RELEASE }}'
111+
- name: Store npm error artefacts
112+
uses: actions/upload-artifact@v3
113+
if: always()
114+
with:
115+
name: npm-logs
116+
path: |
117+
/home/runner/.npm/_logs/**/*
111118
- uses: codecov/codecov-action@v3
112119
continue-on-error: true
113120
with:

.github/workflows/unit-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,9 +37,6 @@ jobs:
3737
- name: Install core dependencies
3838
run: npm ci --no-audit
3939
if: '${{!steps.release-check.outputs.IS_RELEASE}}'
40-
- name: Install site dependencies
41-
run: npm run site:build:install
42-
if: '${{!steps.release-check.outputs.IS_RELEASE}}'
4340
- name: Linting
4441
run: npm run format:ci
4542
if: '${{!steps.release-check.outputs.IS_RELEASE}}'

package.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -65,15 +65,15 @@
6565
"docs": "node ./site/scripts/docs.mjs",
6666
"watch": "c8 --reporter=lcov ava --watch",
6767
"site:build": "run-s site:build:*",
68-
"site:build:install": "cd site && npm ci --no-audit",
68+
"site:build:install": "cd site && npm install --no-audit",
6969
"site:build:assets": "cd site && npm run build",
7070
"postinstall": "node ./scripts/postinstall.mjs",
7171
"prepublishOnly": "node ./scripts/prepare-for-publish.mjs",
7272
"certs": "openssl req -x509 -out localhost.crt -keyout localhost.key -newkey rsa:2048 -nodes -sha256 -subj \"/CN=localhost\" -extensions EXT -config certconf"
7373
},
7474
"config": {
75-
"eslint": "--cache --format=codeframe --max-warnings=0 \"{src,scripts,site,tests,.github}/**/*.{mjs,cjs,js,md,html}\" \"*.{mjs,cjs,js,md,html}\" \".*.{mjs,cjs,js,md,html}\"",
76-
"prettier": "--ignore-path .eslintignore --loglevel=warn \"{src,tools,scripts,site,tests,.github}/**/*.{mjs,cjs,js,md,yml,json,html}\" \"*.{mjs,cjs,js,yml,json,html}\" \".*.{mjs,cjs,js,yml,json,html}\" \"!CHANGELOG.md\" \"!**/*/package-lock.json\" \"!.github/**/*.md\""
75+
"eslint": "--cache --format=codeframe --max-warnings=0 \"{src,scripts,tests,.github}/**/*.{mjs,cjs,js,md,html}\" \"*.{mjs,cjs,js,md,html}\" \".*.{mjs,cjs,js,md,html}\"",
76+
"prettier": "--ignore-path .eslintignore --loglevel=warn \"{src,tools,scripts,tests,.github}/**/*.{mjs,cjs,js,md,yml,json,html}\" \"*.{mjs,cjs,js,yml,json,html}\" \".*.{mjs,cjs,js,yml,json,html}\" \"!CHANGELOG.md\" \"!**/*/package-lock.json\" \"!.github/**/*.md\""
7777
},
7878
"dependencies": {
7979
"@bugsnag/js": "^7.20.0",

site/package-lock.json

Lines changed: 2819 additions & 2316 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

site/package.json

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,11 @@
3333
"lodash.sortby": "^4.7.0",
3434
"prop-types": "^15.7.2",
3535
"react": "^16.14.0",
36+
"react-dom": "^16.14.0",
3637
"react-helmet": "^6.0.0",
3738
"react-instantsearch-dom": "^6.0.0",
3839
"react-router-dom": "^4.3.1",
40+
"react-emotion": "^9.0.0",
3941
"rebass": "^2.3.4",
4042
"styled-components": "^3.4.10",
4143
"styled-system": "^3.2.1"
@@ -45,5 +47,9 @@
4547
"npm-run-all": "^4.1.5",
4648
"sane": "^5.0.0",
4749
"strip-ansi": "^7.0.0"
50+
},
51+
"overrides": {
52+
"react": "$react",
53+
"react-dom": "$react-dom"
4854
}
4955
}

0 commit comments

Comments
 (0)