-
Notifications
You must be signed in to change notification settings - Fork 0
Fi docs style checker #3
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
Conversation
📝 Doc Style Checker Results❌ test-doc.adocError: 🎉 All documentation looks great! No style issues found. Automated by Doc Style Checker • Couchbase Documentation Style Guide |
📝 Doc Style Checker Results❌ test-doc.adocError: 🎉 All documentation looks great! No style issues found. Automated by Doc Style Checker • Couchbase Documentation Style Guide |
📝 Doc Style Checker Results🔍 Summary: 1 issue found across 1 file 📋 test-doc.adoc1 issue found Test CategoryIssue 1Problem: This is a test issue Text: Location: Test location Suggestion: This is a test suggestion Guideline: Test guideline Automated by Doc Style Checker • Couchbase Documentation Style Guide |
📝 Doc Style Checker Results🔍 Summary: 17 issues found across 1 file 📋 test-doc.adoc17 issues found General StyleIssue 1Problem: The heading should be in title case. Text: Location: Heading Suggestion: Data Modeling Guideline: Couchbase Style Guide: General Principles (Capitalization of headings) Issue 2Problem: Avoid using the future tense. Text: Location: Paragraph 1 Suggestion: that you replicate using Sync Gateway. Guideline: Couchbase Style Guide: Write in the Present Tense Issue 3Problem: Use plain language. 'Relating to' can be simplified to 'for'. Text: Location: Paragraph 1 Suggestion: guidance and constraints for the design of data buckets Guideline: Couchbase Style Guide: Use Plain Language Issue 4Problem: Use contractions where appropriate. Text: Location: Paragraph 2 Suggestion: doesn't Guideline: Couchbase Style Guide: Use Plain Language and Contractions Issue 5Problem: Use plain language. 'Encounter' is less direct than 'see'. Text: Location: Section: When you might encounter the error Suggestion: When you might see the error Guideline: Couchbase Style Guide: Use Plain Language Issue 6Problem: The phrase 'the following error' is redundant. The subsequent information indicates an error. Text: Location: Section: For versions 2.x of Sync Gateway, you can encounter the following error: Suggestion: the error: Guideline: Couchbase Style Guide: Use Plain Language Issue 7Problem: Avoid overly formal language. Use a more direct construction. Text: Location: Section: How to avoid the error Suggestion: Avoiding the error Guideline: Couchbase Style Guide: Use Plain Language Issue 8Problem: Use the active voice. The passive voice construction makes the sentence less direct and harder to read. Text: Location: Paragraph 3 Suggestion: Sync Gateway rejects Guideline: Couchbase Style Guide: Use Active Voice Issue 9Problem: Shorten sentences for clarity. Text: Location: Section: Document Structure Suggestion: Couchbase's unit of data is a document. This is the NoSQL equivalent of a row or record. Guideline: Couchbase Style Guide: Use Simple Sentences and Short Paragraphs Issue 10Problem: Use simple sentences and short paragraphs. Break up the sentence for easier readability Text: Location: Section: Value Suggestion: These attachments provide a means to store large media files or any other non-textual data. Couchbase Lite supports attachments of unlimited size. Sync Gateway imposes a 20MB limit for attachments synced to it. Guideline: Couchbase Style Guide: Use Simple Sentences and Short Paragraphs Issue 11Problem: Use the active voice Text: Location: Section: Document Attributes Suggestion: Each document has an ID Guideline: Couchbase Style Guide: Use Active Voice CapitalizationIssue 1Problem: Lowercase 'introduction' as it's a standard section heading. Text: Location: Heading Suggestion: Introduction Guideline: Couchbase Style Guide: General Principles (Capitalization of headings) Issue 2Problem: Capitalize 'Id' when referring to the document key. Text: Location: Section: Value Suggestion: the ID Guideline: Couchbase Style Guide: General Principles (Product names and technical terms) Issue 3Problem: Capitalize 'Id' when referring to the document key. Text: Location: Section: Key Suggestion: the ID Guideline: Couchbase Style Guide: General Principles (Product names and technical terms) Writing StructureIssue 1Problem: The list of 'Document Attributes' could benefit from being presented as a bulleted list for improved readability. Text: Location: Section: Document Attributes Suggestion: Use a bulleted list to present the document attributes. Guideline: Couchbase Style Guide: Use Simple Sentences and Short Paragraphs (Break up text with lists) Google Style GuidelinesIssue 1Problem: Prefer 'for example' over 'e.g.' for clarity. Text: Location: Multiple locations (e.g., Section: Document Attributes) Suggestion: For example, Guideline: Google Developer Style Guide: Abbreviations (Avoid abbreviations that are not widely understood.) ReferencesIssue 1Problem: Incomplete reference; 'Example 1' needs to be expanded to provide more context and clarity. Referencing within the same document is generally discouraged. Text: Location: Property naming Suggestion: see the error details under 'Property prefix error message' Guideline: Writing Concepts + Writing Procedures (linking conventions) Automated by Doc Style Checker • Couchbase Documentation Style Guide |
📝 Doc Style Checker Results🔍 Found 17 style issues in 1 file 📋 test-doc.adocGeneral Style▼ Issue 1 Problem: The heading 'introduction' should be capitalized as 'Introduction'. Headings should be properly capitalized. Text: Location: Heading before the introductory paragraph Suggestion: Change to 'Introduction' Guideline: General principle of clear and consistent writing; implicit in example headings. ▼ Issue 2 Problem: Use present tense. 'They do not necessarily align with constraints' should be rephrased. Text: Location: Introduction, paragraph 1 Suggestion: They don't necessarily align with constraints Guideline: Write in the Present Tense ▼ Issue 3 Problem: Use contractions. 'do not' should be 'don't'. Text: Location: Introduction, paragraph 1 Suggestion: don't Guideline: Use Plain Language and Contractions ▼ Issue 4 Problem: Use contractions. 'does not' should be 'doesn't'. Text: Location: How to avoid the error Suggestion: doesn't Guideline: Use Plain Language and Contractions ▼ Issue 5 Problem: Avoid starting a sentence with 'Any'. Reword the sentence for better flow and clarity. Text: Location: Property Naming, paragraph 2 Suggestion: Sync Gateway rejects any document that matches the reserved property names. See Example 1 for error details. Guideline: Use Simple Sentences and Short Paragraphs; Active Voice (improves flow) ▼ Issue 6 Problem: Prefer 'can' over 'may' when indicating possibility. 'May encounter' should be 'can encounter'. Text: Location: When you might encounter the error Suggestion: You can encounter the error Guideline: Plain language (avoiding overly formal language) ▼ Issue 7 Problem: Avoid starting a sentence with 'For'. Reword the sentence. Text: Location: When you might encounter the error, second bullet Suggestion: In Sync Gateway versions 2.x, you can encounter the following error: Guideline: Use Simple Sentences and Short Paragraphs; Active Voice (improves flow) ▼ Issue 8 Problem: Avoid using phrases like 'How to avoid the error'. Be more direct. Text: Location: Heading Suggestion: Avoiding the error Guideline: Voice and Tone: Aim to keep a neutral tone in the documentation. Think about what drove the user to the documentation and try to be as helpful and to-the-point as possible. Give them what they need to be successful. ▼ Issue 9 Problem: The phrase 'users' data' is awkward. Use 'user data' or 'data'. Text: Location: Document Structure, paragraph 2 Suggestion: user data Guideline: Use Plain Language ▼ Issue 10 Problem: The phrase 'that is' can be simplified to 'that's'. Text: Location: Key, paragraph 4 Suggestion: that's Guideline: Use Plain Language and Contractions ▼ Issue 11 Problem: The phrase 'A JSON value, termed a Document' is awkward. 'Termed' can be removed or rephrased for clarity. Text: Location: Value, paragraph 2 Suggestion: A JSON value (a document). Guideline: Use Plain Language Capitalization▼ Issue 12 Problem: 'couchbase' should be capitalized as 'Couchbase' since it is a product name. Text: Location: Title Suggestion: Couchbase Sync Gateway’s data model Guideline: If Vale flags a product name or other technical industry term as incorrect: If it’s a phrase or single word that must have specific capitalization, add it to the Vocab\Couchbase\accept.txt file. ▼ Issue 13 Problem: 'sync gateway' should be capitalized as 'Sync Gateway' since it is a product name. Text: Location: Title, Introduction, and throughout document Suggestion: Sync Gateway Guideline: If Vale flags a product name or other technical industry term as incorrect: If it’s a phrase or single word that must have specific capitalization, add it to the Vocab\Couchbase\accept.txt file. ▼ Issue 14 Problem: 'Id' should be 'ID'. Text: Location: Document Structure, Key paragraphs, and Value Suggestion: ID Guideline: If Vale flags a product name or other technical industry term as incorrect: If it’s a phrase or single word that must have specific capitalization, add it to the Vocab\Couchbase\accept.txt file. ▼ Issue 15 Problem: 'NOSQL' should be 'NoSQL'. Text: Location: Document Structure, paragraph 1 Suggestion: NoSQL Guideline: If Vale flags a product name or other technical industry term as incorrect: If it’s a phrase or single word that must have specific capitalization, add it to the Vocab\Couchbase\accept.txt file. Writing Structure▼ Issue 16 Problem: Long sentences need to be broken down. 'Search Service indexes are not the same as Index Service indexes. If you deploy the Index Service, Query Service, and Search Service on the same cluster, you can use the Search Service through the N1QL query language.' This sentence is too long and complex. Text: Location: Plain Language and Contractions example (modified from original) Suggestion: Search Service indexes differ from Index Service indexes. When you deploy the Index, Query, and Search Services on the same cluster, you can access Search Service through the N1QL query language. Guideline: Use Simple Sentences and Short Paragraphs Google Style Guidelines▼ Issue 17 Problem: In code samples, the use of curly quotes (“ and ”) is discouraged. Text: Location: Example 1. Property prefix error message Suggestion: {"error":"Bad Request","reason":"user defined top level properties beginning with '_' are not allowed in document body"} Guideline: Google Developer Style Guide: Code samples should use straight quotes. Automated by Doc Style Checker using Couchbase Documentation Style Guide |
📝 Doc Style Checker Results🔍 Found 10 style issues in 1 file 📋 test-doc.adocGeneral Style▼ Issue 1 Problem: Inconsistent use of contractions. The style guide encourages contractions, except for negative constructions, but the content doesn't consistently use them. Text: Location: Introduction Suggestion: Use 'doesn't' instead of 'does not'. Guideline: Use Plain Language and Contractions ▼ Issue 2 Problem: Sentence exceeds 25 words. Text: Location: Introduction Suggestion: Split the sentence into two shorter sentences. For example: 'This page provides guidance and constraints for designing data buckets and documents. These are intended for replication using Sync Gateway.' Guideline: Use Simple Sentences and Short Paragraphs ▼ Issue 3 Problem: Sentence exceeds 25 words. Text: Location: Value Section Suggestion: Split into two sentences: 'Couchbase Lite supports attachments of unlimited size. However, Sync Gateway limits attachments synced to it to 20MB.' Guideline: Use Simple Sentences and Short Paragraphs ▼ Issue 4 Problem: Use present tense. Text: Location: Property Naming Suggestion: listed are rejected Guideline: Write in the Present Tense Capitalization▼ Issue 5 Problem: Inconsistent capitalization of 'Id'. Sometimes it's capitalized, sometimes it's not. Choose one and stick to it. Text: Location: Document Structure Suggestion: Use 'ID' (all caps) or 'id' (all lowercase) consistently. Given the technical context, 'ID' is preferable. Guideline: N/A - Internal Consistency ▼ Issue 6 Problem: The word 'document' should be lowercase when referring to the general concept, and capitalized when it is a specific object. Text: Location: Document Attributes Suggestion: Each document has the following attributes Guideline: N/A - Internal Consistency ▼ Issue 7 Problem: Incorrect capitalization of Couchbase Lite Text: Location: Title Suggestion: Couchbase Sync Gateway Guideline: N/A - Correct Product Name Writing Structure▼ Issue 8 Problem: The introduction lacks a :description: attribute. All concepts and procedures should have one. Text: Location: Introduction Suggestion: Add a :description: attribute above the introduction. Example: Guideline: Writing Concepts & Procedures: Description Attribute ▼ Issue 9 Problem: Missing 'See Also' section. Concepts require a 'See Also' section with related links. Text: Location: End of Document Change History Suggestion: Add a 'See Also' section with links to relevant documentation. Guideline: Writing Concepts: See Also Google Style Guidelines▼ Issue 10 Problem: Use straight quotes instead of curly quotes. Text: Location: Value section Suggestion: users' Guideline: Google Developer Style Guide - Quotes Automated by Doc Style Checker using Couchbase Documentation Style Guide |
📝 Doc Style Checker Results🔍 Found 8 style issues in 1 file 📋 test-doc.adocGeneral Style▼ Issue 1 Problem: Inconsistent tense usage. The document should consistently use the present tense. Text: Location: Introduction Suggestion: that you replicate using Sync Gateway. Guideline: Write in the Present Tense ▼ Issue 2 Problem: Use contractions where appropriate. Text: Location: Introduction Suggestion: doesn't Guideline: Use Plain Language and Contractions ▼ Issue 3 Problem: Sentences are longer than 25 words. Text: Location: When you might encounter the error Suggestion: In Mobile-to-Web Data Sync with Node.js Server SDK and Ottoman.js (the Node.js ODM for Couchbase), the rule conflicts with the _type property that Ottoman.js automatically adds. Guideline: Use Simple Sentences and Short Paragraphs ▼ Issue 4 Problem: Avoid starting a sentence with 'Where'. Text: Location: Section after Example 1 Suggestion: Applicability Guideline: Use Active Voice ▼ Issue 5 Problem: Sentence exceeds the recommended length. Text: Location: Value section, JSON value Suggestion: This JSON object is a collection of key/value pairs. The values can be numbers, strings, arrays, or nested objects. Documents represent complex data structures in a readily parsable and self-organizing way. Guideline: Use Simple Sentences and Short Paragraphs Capitalization▼ Issue 6 Problem: Inconsistent capitalization of 'id'. It should consistently be 'Id' when referring to the document key. Text: Location: Document Structure, Key section Suggestion: the Id Guideline: Follow established capitalization conventions for technical terms ▼ Issue 7 Problem: Incorrect capitalization. Should be 'JSON'. Text: Location: Value section, JSON value Suggestion: a JSON value Guideline: Follow established capitalization conventions for technical terms Writing Structure▼ Issue 8 Problem: The 'How to avoid the error' section could be better structured as a procedure with clear, numbered steps. Text: Location: How to avoid the error Suggestion: To avoid the error, follow these steps:
Guideline: Writing Procedures Automated by Doc Style Checker using Couchbase Documentation Style Guide |
| @@ -0,0 +1,110 @@ | |||
| Data moeeldelling | |||
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.
Automated review comment from Gemini AI:
| Data moeeldelling | |
| Data moeeldelling |
Style Guide Issues Found:
- Couchbase.GeneralStyle (warning) - Use contractions.
Powered by Doc Style Checker using Couchbase Documentation Style Guide
| @@ -0,0 +1,110 @@ | |||
| Data moeeldelling | |||
|
|
|||
| couchbase Sync Gateway’s data model; for secure cloud-to-edge synchronization of enterprise data. | |||
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.
Automated review comment from Gemini AI:
| couchbase Sync Gateway’s data model; for secure cloud-to-edge synchronization of enterprise data. | |
| couchbase Sync Gateway’s data model; for secure cloud-to-edge synchronization of enterprise data. |
Style Guide Issues Found:
- Couchbase.GeneralStyle (error) - Write in the present tense.
- Couchbase.GeneralStyle (warning) - Use contractions.
- Couchbase.Capitalization (error) - Inconsistent capitalization of 'Id'.
- Couchbase.WritingStructure (warning) - Sentence exceeds 25 words.
Powered by Doc Style Checker using Couchbase Documentation Style Guide
|
|
||
| _sync | ||
|
|
||
| _id |
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.
Automated review comment from Gemini AI:
| _id | |
| _id |
Style Guide Issues Found:
- Couchbase.Capitalization (error) - Inconsistent capitalization of 'Id'.
Powered by Doc Style Checker using Couchbase Documentation Style Guide
No description provided.