<p><a class="user-mention" data-hovercard-user-id="301810" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/cbeams">@cbeams</a> just read through this, great doc.</p>
<p>Only 1 suggestion: consider putting the 'Processes' section before 'The Roles Maintainer role' section. The preceding sections discuss elements concerning all roles, the 'Role Maintainer' section only covers that one role, and then 'Processes' goes back to discussing elements that concern all roles.</p>
<p>I see why you did it this way, but switching the two would flow a bit better, I think.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/bisq-network/bisq-docs/pull/46#issuecomment-404890381">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AkpZthK-PgXqh25TKMGzdUI2ybthdlQiks5uGNAygaJpZM4TyopP">mute the thread</a>.<img src="https://github.com/notifications/beacon/AkpZtt9hyM7uHw_jM7ZUUowz81XJ2wcRks5uGNAygaJpZM4TyopP.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/bisq-network/bisq-docs","title":"bisq-network/bisq-docs","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/bisq-network/bisq-docs"}},"updates":{"snippets":[{"icon":"PERSON","message":"@m52go in #46: @cbeams just read through this, great doc.\r\n\r\nOnly 1 suggestion: consider putting the 'Processes' section before 'The Roles Maintainer role' section. The preceding sections discuss elements concerning all roles, the 'Role Maintainer' section only covers that one role, and then 'Processes' goes back to discussing elements that concern all roles.\r\n\r\nI see why you did it this way, but switching the two would flow a bit better, I think."}],"action":{"name":"View Pull Request","url":"https://github.com/bisq-network/bisq-docs/pull/46#issuecomment-404890381"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/bisq-docs/pull/46#issuecomment-404890381",
"url": "https://github.com/bisq-network/bisq-docs/pull/46#issuecomment-404890381",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [bisq-network/bisq-docs] Produce 'Roles' doc (#46)",
"sections": [
{
"text": "",
"activityTitle": "**Steve Jain**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@m52go",
"facts": [

]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueComment\",\n\"repositoryFullName\": \"bisq-network/bisq-docs\",\n\"issueId\": 46,\n\"IssueComment\": \"{{IssueComment.value}}\"\n}"
}
]
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/bisq-network/bisq-docs/pull/46#issuecomment-404890381"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 332040783\n}"
}
],
"themeColor": "26292E"
}
]</script>