Skip to content

Commit ef92914

Browse files
committed
Scaffolding
1 parent a1df675 commit ef92914

File tree

5 files changed

+136
-1
lines changed

5 files changed

+136
-1
lines changed

source/.github/pull_request_template.md renamed to .github/pull_request_template.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Pull Request Info
22

3-
[PR Reviewing Guidelines](https://github.com/mongodb/docs-rust/blob/master/REVIEWING.md)
3+
[PR Reviewing Guidelines](https://github.com/mongodb/docs-php-library/blob/master/REVIEWING.md)
44

55
JIRA - <https://jira.mongodb.org/browse/DOCSP-NNNNN>
66
Staging - <https://docs-mongodbcom-staging.corp.mongodb.com/drivers/docsworker-xlarge/NNNNN/>
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
name: Check Autobuilder for Errors
2+
3+
on:
4+
pull_request:
5+
paths:
6+
- "source/**"
7+
8+
jobs:
9+
check:
10+
runs-on: ubuntu-latest
11+
steps:
12+
- uses: actions/checkout@v3
13+
- uses: cbush/snooty-autobuilder-check@main
Lines changed: 41 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,41 @@
1+
name: Copy Files to docs-shared
2+
3+
on:
4+
workflow_dispatch: {} # use to manually trigger workflow
5+
push:
6+
branches:
7+
- "master"
8+
paths:
9+
- "source/includes/mongodb-compatibility-table-php.rst"
10+
- "source/includes/language-compatibility-table-php.rst"
11+
12+
jobs:
13+
copy-file:
14+
runs-on: ubuntu-latest
15+
steps:
16+
- name: Checkout code
17+
uses: actions/checkout@v3
18+
19+
- name: Copy mongodb-compat table
20+
uses: dmnemec/copy_file_to_another_repo_action@main
21+
env:
22+
API_TOKEN_GITHUB: ${{ secrets.API_TOKEN_GITHUB }}
23+
with:
24+
source_file: "source/includes/mongodb-compatibility-table-php.rst"
25+
destination_repo: "10gen/docs-shared"
26+
destination_folder: "dbx"
27+
user_email: "docs-builder-bot@mongodb.com"
28+
user_name: "docs-builder-bot"
29+
commit_message: "Auto-import from docs-php-library"
30+
31+
- name: Copy language-compat table
32+
uses: dmnemec/copy_file_to_another_repo_action@main
33+
env:
34+
API_TOKEN_GITHUB: ${{ secrets.API_TOKEN_GITHUB }}
35+
with:
36+
source_file: "source/includes/language-compatibility-table-php.rst"
37+
destination_repo: "10gen/docs-shared"
38+
destination_folder: "dbx"
39+
user_email: "docs-builder-bot@mongodb.com"
40+
user_name: "docs-builder-bot"
41+
commit_message: "Auto-import from docs-php-library"

.github/workflows/vale-tdbx.yml

Lines changed: 39 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,39 @@
1+
name: vale-checks
2+
on:
3+
pull_request:
4+
paths:
5+
- "source/**"
6+
7+
jobs:
8+
vale:
9+
name: TDBX Vale rules
10+
runs-on: ubuntu-latest
11+
steps:
12+
- name: checkout
13+
uses: actions/checkout@v4
14+
15+
- id: files
16+
uses: masesgroup/retrieve-changed-files@v2
17+
with:
18+
format: 'csv'
19+
20+
- name: checkout-latest-rules
21+
uses: actions/checkout@v4
22+
with:
23+
repository: mongodb/mongodb-vale-action
24+
path: './tdbx-vale-rules'
25+
token: ${{secrets.GITHUB_TOKEN}}
26+
27+
- name: move-files-for-vale-action
28+
run: |
29+
cp tdbx-vale-rules/.vale.ini .vale.ini
30+
mkdir -p .github/styles/
31+
cp -rf tdbx-vale-rules/.github/styles/ .github/
32+
33+
- name: run-vale
34+
uses: errata-ai/vale-action@reviewdog
35+
with:
36+
reporter: github-pr-check
37+
files: ${{steps.files.outputs.added_modified}}
38+
fail_on_error: true
39+
token: ${{secrets.GITHUB_TOKEN}}

REVIEWING.md

Lines changed: 42 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,42 @@
1+
# Pull Request Reviewing Guidelines for MongoDB PHP Library Documentation
2+
3+
Contributions to the set of documents in this repository can receive reviews from one or both of the following types of reviews:
4+
5+
1. A **copy review**, which focuses on information structure and wording; typically performed by a MongoDB Documentation Team member
6+
2. A **technical review**, which addresses code snippets and the technical correctness of prose; typically performed by a MongoDB engineer.
7+
8+
See the following sections for reviewer expectations for each type of pull request (PR) review:
9+
10+
## Copy Review
11+
12+
Review the structure, wording, and flow of the information in the PR, and correct it if necessary.
13+
14+
### What to Review
15+
16+
- Wording
17+
- Page structure
18+
- Technical content to the extent of the reviewer's understanding.
19+
- Whether the PR fulfills the Acceptance Criteria described in the
20+
linked JIRA ticket.
21+
22+
### What Not to Review
23+
24+
Nothing is completely off-limits to a copy review of a PR -- if you notice a technical issue, it's best to call it out early.
25+
Copy reviewers should constrain their reviews to content within the scope of the JIRA ticket, or otherwise create PRs to address anything unrelated.
26+
27+
## Technical Review
28+
29+
Review the technical accuracy and completeness of a PR and correct it if necessary.
30+
31+
### What to Review
32+
33+
- Code snippets; ensure the code is idiomatic and that all technical claims are correct. e.g. ("To create a `Foo`, use the `Bar.createFoo()` method")
34+
- Problematic explanations that could trip up users who try to follow the documentation.
35+
36+
### What Not to Review
37+
38+
While we welcome any recommendations on wording and structure, avoid blocking approval based on any copy edits. Please entrust the author to make the writing decisions based on style guidelines and team-specific writing conventions, and to create PRs to address anything they deem outside the technical review scope.
39+
40+
- Wording of sentences, although corrections to technical claims are welcome
41+
- Structure of the page
42+
- Any unchanged lines outside the PR unless relevant to the ticket acceptance criteria.

0 commit comments

Comments
 (0)