Add include and exclude fields to Scarb.toml
#644
Assignees
Milestone
Comments
mkaput
added a commit
that referenced
this issue
Oct 3, 2023
This PR makes `scarb package` check for `.scarbignore` files when determining package source files set. This is a trivial addition thanks to features of the `ignore` crate. Supporting this file is effectively a similar feature to the include/exclude fields of Cargo (https://doc.rust-lang.org/cargo/reference/manifest.html#the-exclude-and-include-fields). This PR fixes #644 then, as we do not want to create multiple ways of achieving the same thing. commit-id:3e87fb71
mkaput
added a commit
that referenced
this issue
Oct 3, 2023
This PR makes `scarb package` check for `.scarbignore` files when determining package source files set. This is a trivial addition thanks to features of the `ignore` crate. Supporting this file is effectively a similar feature to the include/exclude fields of Cargo (https://doc.rust-lang.org/cargo/reference/manifest.html#the-exclude-and-include-fields). This PR fixes #644 then, as we do not want to create multiple ways of achieving the same thing. commit-id:3e87fb71
mkaput
added a commit
that referenced
this issue
Oct 3, 2023
This PR makes `scarb package` check for `.scarbignore` files when determining package source files set. This is a trivial addition thanks to features of the `ignore` crate. Supporting this file is effectively a similar feature to the include/exclude fields of Cargo (https://doc.rust-lang.org/cargo/reference/manifest.html#the-exclude-and-include-fields). This PR fixes #644 then, as we do not want to create multiple ways of achieving the same thing. commit-id:3e87fb71
github-merge-queue bot
pushed a commit
that referenced
this issue
Oct 6, 2023
This PR makes `scarb package` check for `.scarbignore` files when determining package source files set. This is a trivial addition thanks to features of the `ignore` crate. Supporting this file is effectively a similar feature to the include/exclude fields of Cargo (https://doc.rust-lang.org/cargo/reference/manifest.html#the-exclude-and-include-fields). This PR fixes #644 then, as we do not want to create multiple ways of achieving the same thing. --- **Stack**: - #758 - #757 - #748 - #746 ⬅⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do not merge manually using the UI - doing so may have unexpected results.*
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Basically implement this: https://doc.rust-lang.org/cargo/reference/manifest.html#the-exclude-and-include-fields
The text was updated successfully, but these errors were encountered: