-
Notifications
You must be signed in to change notification settings - Fork 105
wip: example ToC for maintainer guide #270
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Draft
mefellows
wants to merge
1
commit into
master
Choose a base branch
from
docs/maintainer-guide
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
21 changes: 21 additions & 0 deletions
21
website/docs/implementation_guides/maintainer/checklist/readme.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,21 @@ | ||
| --- | ||
| title: Checklist | ||
| sidebar_label: Checklist | ||
| --- | ||
|
|
||
| # Checklist | ||
|
|
||
| ## Hygiene | ||
|
|
||
| * [ ] Project created from template | ||
| * [ ] Documentation populated from template | ||
| * [ ] Documentation synced via automation to docs.pact.io | ||
| * [ ] Issue templates added | ||
| * [ ] Labels added | ||
| * [ ] Example projects created | ||
| * [ ] Tutorials/workshops to get started | ||
| * [ ] Supports minimum OS/Arch combinations? | ||
|
|
||
| ## Quality | ||
|
|
||
| * [ ] TCK completed and published | ||
10 changes: 10 additions & 0 deletions
10
website/docs/implementation_guides/maintainer/dsl/consumer/readme.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,10 @@ | ||
| --- | ||
| title: Consumer | ||
| sidebar_label: Consumer | ||
| --- | ||
|
|
||
| # Consumer DSL | ||
|
|
||
| - General consumer guidance | ||
| - Matching Rules | ||
| - Specification versions and how this impacts the DSL |
13 changes: 13 additions & 0 deletions
13
website/docs/implementation_guides/maintainer/dsl/http/readme.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| --- | ||
| title: HTTP | ||
| sidebar_label: HTTP | ||
| --- | ||
|
|
||
| # HTTP DSL | ||
|
|
||
| ## Consumer | ||
| TBC | ||
|
|
||
| ## Provider | ||
|
|
||
| TBC |
13 changes: 13 additions & 0 deletions
13
website/docs/implementation_guides/maintainer/dsl/message/readme.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| --- | ||
| title: Message | ||
| sidebar_label: Message | ||
| --- | ||
|
|
||
| # Message DSL | ||
|
|
||
| ## Consumer | ||
| TBC | ||
|
|
||
| ## Provider | ||
|
|
||
| TBC |
13 changes: 13 additions & 0 deletions
13
website/docs/implementation_guides/maintainer/dsl/plugins/readme.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| --- | ||
| title: Plugins | ||
| sidebar_label: Plugins | ||
| --- | ||
|
|
||
| # Plugins DSL | ||
|
|
||
| ## Consumer | ||
| TBC | ||
|
|
||
| ## Provider | ||
|
|
||
| TBC |
16 changes: 16 additions & 0 deletions
16
website/docs/implementation_guides/maintainer/dsl/provider/readme.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,16 @@ | ||
| --- | ||
| title: Provider | ||
| sidebar_label: Provider | ||
| --- | ||
|
|
||
| # Provider DSL | ||
|
|
||
| - General provider guidance e.g. | ||
| - Pact fetching guidance (local, broker etc.) | ||
| - how to setup `pactffi_verifier_set_provider_info` and `pactffi_verifier_add_provider_transport` effectively to support multiple transports | ||
| - state handlers, request filters | ||
| - FFI calls | ||
| - Transports (multiple, how they work, default host/port) | ||
| - State Handlers (setup/teardown, use as hooks, and behaviour if there are no states) | ||
|
|
||
| see https://pact-foundation.slack.com/archives/C02BXLDJ7JR/p1683037785422899 and https://pact-foundation.slack.com/archives/C02BXLDJ7JR/p1683282113939749 for more on this |
12 changes: 12 additions & 0 deletions
12
website/docs/implementation_guides/maintainer/dsl/readme.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,12 @@ | ||
| --- | ||
| title: DSL | ||
| sidebar_label: Introduction | ||
| --- | ||
|
|
||
| # DSL | ||
|
|
||
| DSL Implementation | ||
|
|
||
| Covers Pact nomenclature vs idiomatic naming of things, how to cater for different spec versions, and FFI methods to call. | ||
|
|
||
| *NOTE: It might make sense to split by consumer/provider, rather than use case* |
8 changes: 8 additions & 0 deletions
8
website/docs/implementation_guides/maintainer/example/readme.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| --- | ||
| title: Example | ||
| sidebar_label: Example | ||
| --- | ||
|
|
||
| ## Reference Example | ||
|
|
||
| -> Annotated Pact JS/Pact Node code base |
15 changes: 15 additions & 0 deletions
15
website/docs/implementation_guides/maintainer/ffi/readme.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| --- | ||
| title: FFI | ||
| sidebar_label: FFI | ||
| --- | ||
|
|
||
| ## FFI (Framework? SDK?) | ||
|
|
||
| - Introduction to the Pact Reference implementation / SDK framework / FFI | ||
| - Hello FFI (https://github.com/YOU54F/hello_ffi) | ||
| - Getting started (where to find, download) | ||
| - Ergonomics (make it easy on the end user) | ||
| - General usage, boundaries between FFI and Application | ||
| - Setting up for logging, debugging etc. | ||
| - Identifying the client language | ||
| - Specific methods to call (or should this be spread across the DSL implementation?) |
14 changes: 14 additions & 0 deletions
14
website/docs/implementation_guides/maintainer/general/readme.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,14 @@ | ||
| --- | ||
| title: General | ||
| sidebar_label: General | ||
| --- | ||
|
|
||
| ## General | ||
|
|
||
| - Language / Naming Things / Idioms | ||
| - Minimum supported OS/Arch combinations | ||
| - Logging | ||
| - Debugging | ||
| - Repository hygiene (issue labels, triage etc.) | ||
| - Specification versions | ||
| - A short comparison between the different versions, and how this should be considered in API/DSL design |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,16 @@ | ||
| --- | ||
| title: Introduction | ||
| sidebar_label: Introduction | ||
| --- | ||
|
|
||
| ## Introduction | ||
|
|
||
| - Who it's for | ||
| - Ecosystem view: How it all hangs together (FFI, plugins, client language, CLI tools. Possibly use the diagrams from docs.pact.io/plugins) | ||
| - Maintainer meetings (how to stay in touch. Slack, Zoom monthly etc.) | ||
| - Roadmap (Where to find it, how to contribute to it, RFC process) | ||
| - Criteria for an implementation to be accepted into pact-foundation | ||
| - License should be MIT | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also Apache2
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yeah we have a handful of Apache 2.0 repos. I noted about 9 don't have licenses, which probably should so will flag them for review. |
||
| - 2 maintainers (?) | ||
| - Agree to our community guidelines | ||
| - ...? | ||
9 changes: 9 additions & 0 deletions
9
website/docs/implementation_guides/maintainer/testing/readme.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,9 @@ | ||
| --- | ||
| title: Testing | ||
| sidebar_label: Testing | ||
| --- | ||
|
|
||
| # Testing | ||
|
|
||
| - Expectations for a quality Pact implementation | ||
| - Compatibility suite (should elements of this be threaded throughout the guide rather than its own section?) |
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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a thought for checklist, from previous threads
sustainable CI, or repo pre-reqs, good practises.
pact-foundation/roadmap#8
in addition, docs, ie
etc
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, nice ideas. I just popped a few off the top of my head, but I think it will likely be getting down what we have initially with some "TODOs" that we can populate over time as needed.