Skip to content

docs: improve grammar, clarity, and formatting in README.md - #497

Open
ishikajain-234 wants to merge 1 commit into
numfocus:mainfrom
ishikajain-234:fix-readme-grammar-462
Open

ishikajain-234 wants to merge 1 commit into
numfocus:mainfrom
ishikajain-234:fix-readme-grammar-462

Conversation

@ishikajain-234

@ishikajain-234 ishikajain-234 commented Sep 24, 2026 •

Copy link
Copy Markdown

Description of Changes

This PR addresses the minor grammatical errors, run-on sentences, and formatting inconsistencies in README.md as outlined in #462:

  • Comma splice & run-on sentence: Split the run-on sentence in the introduction into two distinct sentences and standardized the repository name to DISCOVER-Cookbook.
  • Introductory punctuation: Added a missing comma after the introductory dependent clause in the opening reader directions.
  • Awkward phrasing: Simplified and clarified the contribution guidance sentence from "After that issue is made we feel free to open a pull request" to "After that issue is created, feel free to open a pull request".
  • Header capitalization: Standardized ### Bug fixes to ### Bug Fixes.
  • Link casing: Preserved the markdown link while fixing casing: [CONTRIBUTING.md](CONTRIBUTING.md).

These documentation-only changes improve readability and polish without altering any technical requirements or contribution policies.

Related Issue

Fixes #462

Type of Change

  • 🐛 Bug Fix — Fixes a documented issue or incorrect behavior
  • ✨ Content Enhancement — Updates or improves existing content
  • ✍️ New Content — Adds new chapters, examples, or significant sections
  • 🛠️ Other — Please describe in the Description of Changes section

Additional Notes (Optional)

Documentation-only changes confined to the upper sections of root README.md. Verified GitHub-flavored markdown formatting and links.

Checklist

  • I have searched open pull requests to avoid duplicates.
  • I have built the book locally using the build instructions and verified my changes work as expected. (N/A: Changes are confined to the root README.md, which is rendered directly by GitHub rather than Sphinx/Jupyter Book)
  • I have written clear, conventional commit messages, as per the Pull Request Guidelines.

@netlify

netlify Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

❌ Deploy Preview for stupendous-kringle-a86e81 failed.

Name Link
🔨 Latest commit fb4385d
🔍 Latest deploy log https://app.netlify.com/projects/stupendous-kringle-a86e81/deploys/6ab49f0ceb13f8000810db6e

@ishikajain-234

Copy link
Copy Markdown
Author

"Hi! This looks like it's waiting on workflow approval — happy to make any changes if needed. Thanks!"

Fix run-on sentence and comma splice in introduction.
Correct phrasing and capitalization in contribution sections.
Standardize capitalization for CONTRIBUTING.md link and headers.

Fixes numfocus#462

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Minor grammar and clarity improvements in Contribution Guide

1 participant