mirror of
https://github.com/bitwarden/browser.git
synced 2025-01-02 18:17:46 +01:00
[PM-13271] Remove unused ci:coverage from gulpfile (#11455)
Remove the gulp coverage report since we now use jest, and coverage is handled through the root coverage report which is done in a different script.
This commit is contained in:
parent
7c72795d1c
commit
844d2298e9
12
.github/workflows/build-browser.yml
vendored
12
.github/workflows/build-browser.yml
vendored
@ -168,10 +168,6 @@ jobs:
|
|||||||
run: npm run dist:chrome:beta
|
run: npm run dist:chrome:beta
|
||||||
working-directory: browser-source/apps/browser
|
working-directory: browser-source/apps/browser
|
||||||
|
|
||||||
- name: Gulp
|
|
||||||
run: gulp ci
|
|
||||||
working-directory: browser-source/apps/browser
|
|
||||||
|
|
||||||
- name: Upload Opera artifact
|
- name: Upload Opera artifact
|
||||||
uses: actions/upload-artifact@50769540e7f4bd5e21e526ee35c689e35e0d6874 # v4.4.0
|
uses: actions/upload-artifact@50769540e7f4bd5e21e526ee35c689e35e0d6874 # v4.4.0
|
||||||
with:
|
with:
|
||||||
@ -235,14 +231,6 @@ jobs:
|
|||||||
path: browser-source.zip
|
path: browser-source.zip
|
||||||
if-no-files-found: error
|
if-no-files-found: error
|
||||||
|
|
||||||
- name: Upload coverage artifact
|
|
||||||
if: false
|
|
||||||
uses: actions/upload-artifact@50769540e7f4bd5e21e526ee35c689e35e0d6874 # v4.4.0
|
|
||||||
with:
|
|
||||||
name: coverage-${{ env._BUILD_NUMBER }}.zip
|
|
||||||
path: browser-source/apps/browser/coverage/coverage-${{ env._BUILD_NUMBER }}.zip
|
|
||||||
if-no-files-found: error
|
|
||||||
|
|
||||||
build-safari:
|
build-safari:
|
||||||
name: Build Safari
|
name: Build Safari
|
||||||
runs-on: macos-13
|
runs-on: macos-13
|
||||||
|
@ -14,7 +14,6 @@ const betaBuild = process.env.BETA_BUILD === "1";
|
|||||||
const paths = {
|
const paths = {
|
||||||
build: "./build/",
|
build: "./build/",
|
||||||
dist: "./dist/",
|
dist: "./dist/",
|
||||||
coverage: "./coverage/",
|
|
||||||
node_modules: "./node_modules/",
|
node_modules: "./node_modules/",
|
||||||
popupDir: "./src/popup/",
|
popupDir: "./src/popup/",
|
||||||
cssDir: "./src/popup/css/",
|
cssDir: "./src/popup/css/",
|
||||||
@ -276,17 +275,6 @@ function stdOutProc(proc) {
|
|||||||
proc.stderr.on("data", (data) => console.error(data.toString()));
|
proc.stderr.on("data", (data) => console.error(data.toString()));
|
||||||
}
|
}
|
||||||
|
|
||||||
async function ciCoverage(cb) {
|
|
||||||
const { default: zip } = await import("gulp-zip");
|
|
||||||
const { default: filter } = await import("gulp-filter");
|
|
||||||
|
|
||||||
return gulp
|
|
||||||
.src(paths.coverage + "**/*")
|
|
||||||
.pipe(filter(["**", "!coverage/coverage*.zip"]))
|
|
||||||
.pipe(zip(`coverage${buildString()}.zip`))
|
|
||||||
.pipe(gulp.dest(paths.coverage));
|
|
||||||
}
|
|
||||||
|
|
||||||
function applyBetaLabels(manifest) {
|
function applyBetaLabels(manifest) {
|
||||||
manifest.name = "Bitwarden Password Manager BETA";
|
manifest.name = "Bitwarden Password Manager BETA";
|
||||||
manifest.short_name = "Bitwarden BETA";
|
manifest.short_name = "Bitwarden BETA";
|
||||||
@ -320,5 +308,3 @@ exports["dist:safari:mas"] = distSafariMas;
|
|||||||
exports["dist:safari:masdev"] = distSafariMasDev;
|
exports["dist:safari:masdev"] = distSafariMasDev;
|
||||||
exports["dist:safari:dmg"] = distSafariDmg;
|
exports["dist:safari:dmg"] = distSafariDmg;
|
||||||
exports.dist = gulp.parallel(distFirefox, distChrome, distOpera, distEdge);
|
exports.dist = gulp.parallel(distFirefox, distChrome, distOpera, distEdge);
|
||||||
exports["ci:coverage"] = ciCoverage;
|
|
||||||
exports.ci = ciCoverage;
|
|
||||||
|
@ -26,7 +26,6 @@
|
|||||||
"dist:safari:masdev": "npm run build:prod && gulp dist:safari:masdev",
|
"dist:safari:masdev": "npm run build:prod && gulp dist:safari:masdev",
|
||||||
"dist:safari:dmg": "npm run build:prod && gulp dist:safari:dmg",
|
"dist:safari:dmg": "npm run build:prod && gulp dist:safari:dmg",
|
||||||
"test": "jest",
|
"test": "jest",
|
||||||
"test:coverage": "jest --coverage --coverageDirectory=coverage",
|
|
||||||
"test:watch": "jest --watch",
|
"test:watch": "jest --watch",
|
||||||
"test:watch:all": "jest --watchAll"
|
"test:watch:all": "jest --watchAll"
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user