Skip to content
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.

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

v7: Port to TypeScript, provide hybrid cjs/esm interface #408

Merged
merged 12 commits into from
Apr 10, 2024
Merged

Conversation

isaacs
Copy link
Owner

@isaacs isaacs commented Apr 10, 2024

No description provided.

The `noChmod` option is deprecated, and replaced with the `chmod`
option. The default is now to *not* explicitly set the mode of created
filesystem entries.

This is a deviation from `tar(1)` implementations, but prevents the need
to call `process.umask()`, which is deprecated and not thread safe.

To still set modes explicitly, while avoiding the need to call
`process.umask()`, a `processUmask` numeric option may now be provided
along with `chmod: true`.
This replaces a lot of manual mkdir/rimraf with t.testdir()
tar-stream creates some interesting tarballs, but they are technically
allowed, and should be handled properly.

Fix: #312

Also, this cleans up a flaky race condition in the unpack test.
@isaacs isaacs merged commit 957da75 into main Apr 10, 2024
12 checks passed
@isaacs isaacs deleted the isaacs/ts branch April 10, 2024 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants