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

took the functionality of the third party subcommand from the list_commands function #15075

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

ibilalkayy
Copy link
Contributor

Hey there,

I just moved the functionality of the third-party subcommand from the list_commands function to another new function by the name of third_party_subcommand name and called that function in the list_commands function.

From my understanding regarding the third-party subcommand from this tracking issue, following points should be performed.

  • The code that gathers third-party subcommands in list_commands should be moved into a separate function that I did.

  • This new function will be called both by list_commands and the code for adding subcommand completions. Although I called the function in the list_commands but didn't understand the second point.

  • Test the change that I made in the first place but it gave me this error:

carbon (68)

@rustbot
Copy link
Collaborator

rustbot commented Jan 16, 2025

r? @ehuss

rustbot has assigned @ehuss.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added A-cli Area: Command-line interface, option parsing, etc. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 16, 2025
@ibilalkayy ibilalkayy force-pushed the third_party_subcommand branch from 31ab24d to ea83afd Compare January 16, 2025 22:49
@ibilalkayy
Copy link
Contributor Author

r? @epage this is the PR.

@rustbot rustbot assigned epage and unassigned ehuss Jan 16, 2025
Copy link
Contributor

@epage epage left a comment

Choose a reason for hiding this comment

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

Thanks!

@epage epage enabled auto-merge January 16, 2025 22:55
@epage epage added this pull request to the merge queue Jan 16, 2025
Merged via the queue into rust-lang:master with commit 531215f Jan 16, 2025
21 checks passed
@ibilalkayy ibilalkayy deleted the third_party_subcommand branch January 16, 2025 23:59
bors added a commit to rust-lang-ci/rust that referenced this pull request Jan 18, 2025
Update cargo

13 commits in 088d496082726091024f1689c124a0c3dccbd775..045bf21b36a2e1f3ed85e38278d1c3cc4305e134
2025-01-10 20:10:21 +0000 to 2025-01-17 14:59:36 +0000
- created a function for user defined aliases (rust-lang/cargo#15076)
- took the functionality of the third party subcommand from the list_commands function (rust-lang/cargo#15075)
- fix: wrong concat and field name (rust-lang/cargo#15074)
- fix(publish): Report all unpublishable packages  (rust-lang/cargo#15070)
- docs(cargo-clippy): correct typo (rust-lang/cargo#15072)
- docs(cargo-package): alwasy include the lockfile (rust-lang/cargo#15067)
- docs(ref): Deprecate 'package.authors'  (rust-lang/cargo#15068)
- fix(build-std): parse as comma-separated list (rust-lang/cargo#15065)
- Fix benchsuite issue with newer versions of git (rust-lang/cargo#15069)
- Document that cargo automatically registers variables used in env! macro to trigger rebuilds (rust-lang/cargo#15062)
- perf(cargo-package): match certain path prefix with pathspec (rust-lang/cargo#14997)
- Clarify note in example (rust-lang/cargo#15054)
- chore(deps): update msrv (3 versions) to v1.82 (rust-lang/cargo#15050)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-cli Area: Command-line interface, option parsing, etc. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants