Add command stubs for init, new, start, pack with project detection#4858
Open
liliankasem wants to merge 1 commit intovnextfrom
Open
Add command stubs for init, new, start, pack with project detection#4858liliankasem wants to merge 1 commit intovnextfrom
liliankasem wants to merge 1 commit intovnextfrom
Conversation
Register init, new, start, and pack commands with their CLI options. Commands display workload install guidance until workloads are available. Includes ProjectDetector for runtime detection from project files. 47 tests passing.
This was referenced Apr 20, 2026
b6f57b0 to
2afd863
Compare
satvu
reviewed
Apr 20, 2026
|
|
||
| /// <summary> | ||
| /// Launches the Azure Functions host runtime. Supports 'func start' and | ||
| /// Placeholder for 'func start' (not yet implemented). |
Member
There was a problem hiding this comment.
Confused by this comment - supports func start and placeholder for func start? What is this placeholder?
aishwaryabh
reviewed
Apr 20, 2026
| _interaction.WriteMarkupLine( | ||
| "[grey]Install a workload to initialize a project:[/]"); | ||
| _interaction.WriteBlankLine(); | ||
| _interaction.WriteMarkupLine(" [white]func workload install dotnet[/] [grey]C#, F#[/]"); |
Contributor
There was a problem hiding this comment.
can we store a map of worker to language options? i see this is being reused here and in newCommand so it would be nice to have a central map so we it's easy to add new workers in the future too
aishwaryabh
reviewed
Apr 20, 2026
|
|
||
| private readonly IInteractionService _interaction; | ||
|
|
||
| public PackCommand(IInteractionService interaction) |
Contributor
There was a problem hiding this comment.
in this design, how are we accounting for different subcommands (like --skip-install for node)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Register init, new, start, and pack commands with their CLI options. Commands display workload install guidance until workloads are available. Includes ProjectDetector for runtime detection from project files. 47 tests passing.
Issue describing the changes in this PR
resolves #issue_for_this_pr
Pull request checklist
release_notes.mdAdditional information
Additional PR information