Skip to content

Docs: Inline CommandArgument required/optional style in template parameter docs#1090

Merged
patriksvensson merged 1 commit intospectreconsole:mainfrom
ArveSystad:commandargument-inline-docs
Nov 25, 2022
Merged

Docs: Inline CommandArgument required/optional style in template parameter docs#1090
patriksvensson merged 1 commit intospectreconsole:mainfrom
ArveSystad:commandargument-inline-docs

Conversation

@ArveSystad
Copy link
Contributor

Found myself looking this up every time I wrote a new command, so it might as well be part of what my IDE shows me while writing.

@dnfadmin
Copy link

dnfadmin commented Nov 24, 2022

CLA assistant check
All CLA requirements met.

Copy link
Contributor

Choose a reason for hiding this comment

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

Awesome, but it's the other way around.

<> are required, [] are optional.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed!

@ArveSystad ArveSystad force-pushed the commandargument-inline-docs branch from 06528e3 to 84f7bb5 Compare November 25, 2022 07:37
@ArveSystad ArveSystad force-pushed the commandargument-inline-docs branch from 84f7bb5 to 8ec61b5 Compare November 25, 2022 07:39
@patriksvensson
Copy link
Contributor

@ArveSystad You will need to sign the CLA as well.

@ArveSystad
Copy link
Contributor Author

@ArveSystad You will need to sign the CLA as well.

Yea, I know. Keep getting errors from that signing app since yesterday evening: MongoTopologyClosedError: Topology is closed. I'll try again a little later.

@patriksvensson
Copy link
Contributor

I've raised an issue over at dotnet-foundation/projects#236.

Hopefully it should be resolved soon. I will go ahead and merge your change anyway since its a small change.

@patriksvensson patriksvensson merged commit 8398234 into spectreconsole:main Nov 25, 2022
@patriksvensson
Copy link
Contributor

Merged! Thank you for your contribution. Much appreciated! 👍

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.

3 participants

Comments