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

Copy fixtures to temp dir #211

Merged
merged 5 commits into from
Jul 26, 2018

Conversation

fitzgen
Copy link
Member

@fitzgen fitzgen commented Jul 14, 2018

Make sure these boxes are checked! 📦✅

  • You have the latest version of rustfmt installed and have your
    cloned directory set to nightly
$ rustup override set nightly
$ rustup component add rustfmt-preview --toolchain nightly
  • You ran rustfmt on the code base before submitting
  • You reference which issue is being closed in the PR text

✨✨ 😄 Thanks so much for contributing to wasm-pack! 😄 ✨✨

Copy link
Member

@ashleygwilliams ashleygwilliams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hrm this looks like it is still failing. gonna hold off on sneaking this into 0.4.1 because it's 8pm and it's the weekend now <3

@migerh
Copy link
Contributor

migerh commented Jul 14, 2018

Just wanted to leave a quick note that tempdir is deprecated in favor of tempfile: rust-lang-deprecated/tempdir#45

@fitzgen fitzgen force-pushed the copy-fixtures-to-temp-dir branch 5 times, most recently from 526a8cb to c5b528b Compare July 16, 2018 20:45
@fitzgen
Copy link
Member Author

fitzgen commented Jul 16, 2018

r? @ashleygwilliams -- this should be ready for your attention again!

@ashleygwilliams
Copy link
Member

@fitzgen after #220 this will need a rebase- otherwise i think it looks good to me and i'll merge :)

If a test is going to do a potentially destructive operation, instead of
mutating shared fixtures, it should copy the fixture to its own unique directory
and then mutate that copy (doesn't this sound like Rust's borrowing and
ownership rules?! ^.^)
This reverts commit 24c0ca7.

That commit is no longer necessary after 10ed46d.
The `tempdir` crate is deprecated and its functionality has been folded into the
`tempfile` crate.
The implementation relies on the crate directory existing, and the `pkg`
directory in the crate directory existing as well.
@fitzgen
Copy link
Member Author

fitzgen commented Jul 25, 2018

Rebased!

@ashleygwilliams ashleygwilliams merged commit 56f1896 into rustwasm:master Jul 26, 2018
@fitzgen fitzgen deleted the copy-fixtures-to-temp-dir branch July 26, 2018 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants