Skip to content

Commit

Permalink
Merge branch 'master' into workspace
Browse files Browse the repository at this point in the history
  • Loading branch information
beyondkmp authored Oct 10, 2024
2 parents 6370376 + dfa35c3 commit a2871a1
Show file tree
Hide file tree
Showing 13 changed files with 1,633 additions and 2,782 deletions.
5 changes: 5 additions & 0 deletions .changeset/lucky-tigers-do.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"app-builder-lib": patch
---

fix: add additional default exclusions to copy logic
5 changes: 5 additions & 0 deletions .changeset/real-cougars-watch.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"app-builder-lib": patch
---

chore(deps): update @electron/rebuild to v3.7.0
2 changes: 1 addition & 1 deletion packages/app-builder-lib/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@
"@develar/schema-utils": "~2.6.5",
"@electron/notarize": "2.5.0",
"@electron/osx-sign": "1.3.1",
"@electron/rebuild": "3.6.1",
"@electron/rebuild": "3.7.0",
"@electron/universal": "2.0.1",
"@malept/flatpak-bundler": "^0.4.0",
"@types/fs-extra": "9.0.13",
Expand Down
2 changes: 1 addition & 1 deletion packages/app-builder-lib/src/electron/electronVersion.ts
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ export async function computeElectronVersion(projectDir: string): Promise<string

const potentialRootDirs = [projectDir, await getProjectRootPath(projectDir)]
let dependency: NameAndVersion | null = null
for await (const dir of potentialRootDirs) {
for (const dir of potentialRootDirs) {
const metadata = await orNullIfFileNotExist(readJson(path.join(dir, "package.json")))
dependency = metadata ? findFromPackageMetadata(metadata) : null
if (dependency) {
Expand Down
4 changes: 2 additions & 2 deletions packages/app-builder-lib/src/fileMatcher.ts
Original file line number Diff line number Diff line change
Expand Up @@ -15,13 +15,13 @@ export const excludedNames =
".git,.hg,.svn,CVS,RCS,SCCS," +
"__pycache__,.DS_Store,thumbs.db,.gitignore,.gitkeep,.gitattributes,.npmignore," +
".idea,.vs,.flowconfig,.jshintrc,.eslintrc,.circleci," +
".yarn-integrity,.yarn-metadata.json,yarn-error.log,yarn.lock,package-lock.json,npm-debug.log," +
".yarn-integrity,.yarn-metadata.json,yarn-error.log,yarn.lock,package-lock.json,npm-debug.log,pnpm-lock.yaml," +
"appveyor.yml,.travis.yml,circle.yml,.nyc_output,.husky,.github,electron-builder.env"

export const excludedExts =
"iml,hprof,orig,pyc,pyo,rbc,swp,csproj,sln,suo,xproj,cc,d.ts," +
// https://github.com/electron-userland/electron-builder/issues/7512
"mk,a,o,forge-meta"
"mk,a,o,obj,forge-meta"

function ensureNoEndSlash(file: string): string {
if (path.sep !== "/") {
Expand Down
17 changes: 13 additions & 4 deletions packages/app-builder-lib/src/util/NodeModuleCopyHelper.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,19 @@ import { NodeModuleInfo } from "./packageDependencies"
import { realpathSync } from "fs"

const excludedFiles = new Set(
[".DS_Store", "node_modules" /* already in the queue */, "CHANGELOG.md", "ChangeLog", "changelog.md", "Changelog.md", "Changelog", "binding.gyp", ".npmignore"].concat(
excludedNames.split(",")
)
)
[
".DS_Store",
"node_modules" /* already in the queue */,
"CHANGELOG.md",
"ChangeLog",
"changelog.md",
"Changelog.md",
"Changelog",
"binding.gyp",
".npmignore",
"node_gyp_bins",
].concat(excludedNames.split(",")))

Check warning on line 24 in packages/app-builder-lib/src/util/NodeModuleCopyHelper.ts

View workflow job for this annotation

GitHub Actions / test-linux (ArtifactPublisherTest,BuildTest,ExtraBuildTest,RepoSlugTest,binDownloadTest,configura...

Insert `⏎`

Check warning on line 24 in packages/app-builder-lib/src/util/NodeModuleCopyHelper.ts

View workflow job for this annotation

GitHub Actions / test-linux (snapTest,debTest,fpmTest,protonTest)

Insert `⏎`

const topLevelExcludedFiles = new Set([
"karma.conf.js",
".coveralls.yml",
Expand Down
Loading

0 comments on commit a2871a1

Please sign in to comment.