Skip to content

Allow 'code' commands to accept multiple file arguments#816

Merged
dtivel merged 1 commit intodotnet:mainfrom
Faithlife:multiple-files
Apr 22, 2025
Merged

Allow 'code' commands to accept multiple file arguments#816
dtivel merged 1 commit intodotnet:mainfrom
Faithlife:multiple-files

Conversation

@bgrainger
Copy link
Contributor

Fixes #815

The command documentation specifies that these commands take a "file(s)" parameter. They currently accept only a single argument, although it can be a glob to specify multiple files. This changes them to accept multiple filenames as separate arguments, similar to the existing behavior of signtool and azuresigntool.

@dtivel
Copy link
Collaborator

dtivel commented Apr 21, 2025

@bgrainger, this change looks good. Can you please rebase on latest and update?

…#815

The command documentation specifies that these commands take a "file(s)" parameter. They currently accept only a single argument, although it can be a glob to specify multiple files. This changes them to accept multiple filenames as separate arguments, similar to the existing behavior of signtool and azuresigntool.
@bgrainger
Copy link
Contributor Author

Rebased.

@dtivel dtivel merged commit 7cdfc01 into dotnet:main Apr 22, 2025
3 checks passed
@dtivel
Copy link
Collaborator

dtivel commented Apr 24, 2025

@bgrainger, this should be available in 0.9.1-beta.25224.2.

@bgrainger bgrainger deleted the multiple-files branch April 25, 2025 16:09
bgrainger added a commit to Faithlife/FaithlifeBuild that referenced this pull request Jan 12, 2026
The upstream repo has fixed the two issues we created a fork for:

- Accept multiple file names: dotnet/sign#816
- Use just Azure CLI credential: dotnet/sign#891
bgrainger added a commit to Faithlife/FaithlifeBuild that referenced this pull request Jan 12, 2026
The upstream repo has fixed the two issues we created a fork for:

- Accept multiple file names: dotnet/sign#816
- Use just Azure CLI credential: dotnet/sign#891
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"trusted-signing <file(s)>" doesn't accept multiple files

3 participants