Set umask once #7176

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

Jarred-Sumner merged 1 commit into main from jarred/umask

Nov 17, 2023

Collaborator

Jarred-Sumner commented Nov 17, 2023

What does this PR do?

Set umask once

How did you verify your code works?

I didn't


          Set umask once

2e6a087

Jarred-Sumner merged commit 711ea16 into main

Jarred-Sumner deleted the jarred/umask branch

November 17, 2023 13:08

Contributor

github-actions bot commented Nov 17, 2023

❌ @Jarred-Sumner 2 files with test failures on linux-x64-baseline:

test/cli/install/registry/bun-install-registry.test.ts
test/js/node/v8/capture-stack-trace.test.js

View test output

^{#2fced9b1e67272debc2b35876649536c6a09a757}

Contributor

github-actions bot commented Nov 17, 2023

❌ @Jarred-Sumner 2 files with test failures on linux-x64:

test/cli/install/registry/bun-install-registry.test.ts
test/js/node/v8/capture-stack-trace.test.js

View test output

^{#2fced9b1e67272debc2b35876649536c6a09a757}

Contributor

github-actions bot commented Nov 17, 2023

❌ @Jarred-Sumner 8 files with test failures on bun-darwin-aarch64:

test/cli/install/registry/bun-install-registry.test.ts
test/integration/next/default-pages-dir/test/dev-server.test.ts
test/js/bun/dns/resolve-dns.test.ts
test/js/bun/spawn/spawn-streaming-stdin.test.ts
test/js/bun/test/test-test.test.ts
test/js/bun/util/which.test.ts
test/js/node/v8/capture-stack-trace.test.js
test/js/third_party/jsonwebtoken/claim-nbf.test.js

View test output

^{#2fced9b1e67272debc2b35876649536c6a09a757}

Contributor

github-actions bot commented Nov 17, 2023

❌ @Jarred-Sumner 8 files with test failures on bun-darwin-x64-baseline:

test/cli/install/registry/bun-install-registry.test.ts
test/js/bun/spawn/spawn-streaming-stdout.test.ts
test/js/bun/util/which.test.ts
test/js/node/fs/fs.test.ts
test/js/node/v8/capture-stack-trace.test.js
test/js/node/watch/fs.watchFile.test.ts
test/js/third_party/webpack/webpack.test.ts
test/js/web/timers/setTimeout.test.js

View test output

^{#2fced9b1e67272debc2b35876649536c6a09a757}

ryoppippi pushed a commit to ryoppippi/bun that referenced this pull request


          Set umask once (oven-sh#7176)

Co-authored-by: Jarred Sumner <[email protected]>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet