-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
feat: Initial support for single-file packages #12245
Merged
Merged
Commits on Jun 9, 2023
-
Configuration menu - View commit details
-
Copy full SHA for 87e128b - Browse repository at this point
Copy the full SHA 87e128bView commit details -
Configuration menu - View commit details
-
Copy full SHA for c0dd8ae - Browse repository at this point
Copy the full SHA c0dd8aeView commit details -
Configuration menu - View commit details
-
Copy full SHA for 21736ed - Browse repository at this point
Copy the full SHA 21736edView commit details -
Configuration menu - View commit details
-
Copy full SHA for b2b4d97 - Browse repository at this point
Copy the full SHA b2b4d97View commit details -
feat(cli): Define precedence among subcommands
I decided to start things off fairly simple. It either looks like the user is specifying a manifest or they aren't. This is assuming we only need to handle `cargo foo.rs` and `cargo ./foo.rs` and not `cargo foo`. I think in most shebang cases, path multiple components will present which makes that a dead giveaway and likely to not overlap with aliases and subcommands. For reference, dlang's dub goes a lot further 1. Checks for the subcommand name being `-` 2. Checks if subcommand name ends with `.d` 3. Checks if subcommand name is built-in 4. Checks if a file with the subcommand name exists 5. Checks if a file with the subcommand name + `.d` exists This would allow a `run.d` to override `dub-run` which doesn't seem good.
Configuration menu - View commit details
-
Copy full SHA for 1a30fc8 - Browse repository at this point
Copy the full SHA 1a30fc8View commit details
Commits on Jun 10, 2023
-
fix(cli): Warn on stable for precedence changes
This will give us a window to collect feedback on if this affects anyone.
Configuration menu - View commit details
-
Copy full SHA for 3c15d24 - Browse repository at this point
Copy the full SHA 3c15d24View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2bd9f14 - Browse repository at this point
Copy the full SHA 2bd9f14View commit details -
Configuration menu - View commit details
-
Copy full SHA for c421e0b - Browse repository at this point
Copy the full SHA c421e0bView commit details -
feat(cli): Pull in cargo-script-mvs core logic
This is no where near the implementation we want but I think we should develop incrementally on top of what we already have. See https://github.com/epage/cargo-script-mvs/tree/main
Configuration menu - View commit details
-
Copy full SHA for 33c9d8e - Browse repository at this point
Copy the full SHA 33c9d8eView commit details -
docs(unstable): Expand on manifest commands so far
This is written to reflect the current implementation though some parts might read a little weird because I didn't want to write throw-away documentation for when we change this. For example, single-file packages are currently only supported in `cargo <command>` and not as manifest paths but this will change.
Configuration menu - View commit details
-
Copy full SHA for 6b0b5a8 - Browse repository at this point
Copy the full SHA 6b0b5a8View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.