Skip to content

Conversation

captainsafia
Copy link
Member

Add some helpful docs on how to use the darc command-line tool.

@captainsafia captainsafia added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Jul 31, 2020
Copy link
Contributor

@dougbu dougbu left a comment

Choose a reason for hiding this comment

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

Just nits and minor cleanup. I don't need to look again.


Subscriptions are objects that define the ecosystem repos we are listening for updates to, the frequency we are looking for updates, and more.

```
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: VS Code will complain about not specifying the type of code block. This is pretty generic so bash or ps1 would both work. If you do this, remove the leading $s and do it for all the scripting you're adding.


Note: Merge policies can only be set on unbatched subscriptions. Be sure to set/unset the `Merge Policies` field properly as you toggle batchability.


Copy link
Contributor

Choose a reason for hiding this comment

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

nit: Only need one final newline 😈

captainsafia and others added 2 commits July 31, 2020 13:34
Co-authored-by: Doug Bunting <6431421+dougbu@users.noreply.github.com>
@ghost
Copy link

ghost commented Jul 31, 2020

Hello @captainsafia!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

Do note that I've been instructed to only help merge pull requests of this repository that have been opened for at least 60 minutes, a condition that will be fulfilled in about 6 minutes. No worries though, I will be back when the time is right! 😉

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit 723e32a into master Jul 31, 2020
@ghost ghost deleted the safia/darc-docs branch July 31, 2020 21:49
HaoK pushed a commit that referenced this pull request Aug 7, 2020
* Update documentation on darc

* Apply suggestions from code review

Co-authored-by: Doug Bunting <6431421+dougbu@users.noreply.github.com>

* Update code blocks in referenced assemblies

Co-authored-by: Doug Bunting <6431421+dougbu@users.noreply.github.com>
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants