Skip to content

docs: improve package READMEs with quick start guides & detailed features#462

Merged
divyanshub024 merged 2 commits intodevfrom
dv/readme
Mar 2, 2026
Merged

docs: improve package READMEs with quick start guides & detailed features#462
divyanshub024 merged 2 commits intodevfrom
dv/readme

Conversation

@divyanshub024
Copy link
Member

@divyanshub024 divyanshub024 commented Mar 2, 2026

Description

improve package READMEs with quick start guides & detailed features

Type of Change

  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Code refactor
  • Build configuration change
  • Documentation
  • Chore

Summary by CodeRabbit

  • Documentation
    • Overhauled example and package READMEs with clearer titles, expanded descriptions, and explicit "What this demonstrates" and "How to run" quick-starts.
    • Added step-by-step run commands, platform-specific install/run instructions, and CLI quick-start workflows.
    • Introduced package and license badges and clarified usage/feature explanations across examples and packages.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 2, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 28f5619 and e97bb6f.

📒 Files selected for processing (2)
  • packages/stac/README.md
  • packages/stac_logger/analysis_options.yaml
🚧 Files skipped from review as they are similar to previous changes (1)
  • packages/stac/README.md

📝 Walkthrough

Walkthrough

This PR updates README content across examples and packages: rewrites descriptions to emphasize Server-Driven UI and Dart DSL, standardizes badges, and adds expanded run/install/Quick Start instructions and environment notes; no code or public API changes.

Changes

Cohort / File(s) Summary
Example READMEs
examples/counter_example/README.md, examples/movie_app/README.md, examples/stac_gallery/README.md
Rewrote headers and intros to emphasize SDUI/Dart DSL, replaced "Getting Started" with "What this demonstrates" and "How to run", added step-by-step run instructions and concrete flutter commands.
Stac package README
packages/stac/README.md
Major content rewrite focusing on Stac's Dart DSL, added "Why use Stac?" and an expanded Quick Start with inline code examples and usage guidance; presentation-first changes, no API edits.
CLI README & env docs
packages/stac_cli/README.md
Expanded CLI README: added install snippets for macOS/Linux and Windows, extended Quick Start CLI workflow, and an Environment section listing credential locations and required env keys.
Other package READMEs
packages/stac_core/README.md, packages/stac_framework/README.md, packages/stac_logger/README.md, packages/stac_webview/README.md
Added Pub and MIT license badges and minor wording/capitalization tweaks to README headers and descriptions.
Lint configuration
packages/stac_logger/analysis_options.yaml
Switched analysis options include from flutter_lints to recommended Dart lints (changed lint include reference).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested reviewers

  • Potatomonsta
  • rahulbisht25

Poem

🐰 I hop through README fields so bright,
Polishing guides by moonlit night.
Badges sparkle, examples sing,
Quick-starts ready — give them wing! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and comprehensively describes the main changes: updates to README files across multiple packages with additions of quick start guides and expanded feature documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dv/readme

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 3

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@packages/stac_cli/README.md`:
- Around line 9-17: The README.md currently shows unsafe curl|bash and irm|iex
install examples; replace them with explicit download + integrity verification +
explicit execution instructions: for macOS/Linux update the macOS / Linux block
to download install.sh (referencing install.sh), instruct users to verify
checksum/signature, then run it with bash install.sh; for Windows replace the
PowerShell pipe example with Invoke-WebRequest to save install.ps1 (referencing
install.ps1), instruct users to verify checksum/signature, then run it with
powershell -ExecutionPolicy Bypass -File .\install.ps1; keep the steps concise
and clearly label verification before execution.

In `@packages/stac/README.md`:
- Line 151: Update the user-facing label value where it's misspelled: change the
text property currently set as 'Procced' to the correct spelling 'Proceed'
(modify the example entry with text: 'Procced' in the README). Ensure the
correction is applied to the same example key (text) so the displayed button
label reads "Proceed".
- Around line 59-64: The quick-start snippet references defaultStacOptions but
the import is commented out, causing a runtime/name error; update the example so
it runs as-is by either uncommenting the import that provides defaultStacOptions
or by showing inline/optional usage of Stac.initialize (e.g., call
Stac.initialize() without options or define defaultStacOptions in the snippet).
Ensure the README example consistently defines or imports the symbol referenced
by Stac.initialize to avoid copy/paste failure.

ℹ️ Review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0764435 and 28f5619.

⛔ Files ignored due to path filters (1)
  • assets/login.png is excluded by !**/*.png
📒 Files selected for processing (9)
  • examples/counter_example/README.md
  • examples/movie_app/README.md
  • examples/stac_gallery/README.md
  • packages/stac/README.md
  • packages/stac_cli/README.md
  • packages/stac_core/README.md
  • packages/stac_framework/README.md
  • packages/stac_logger/README.md
  • packages/stac_webview/README.md

@divyanshub024 divyanshub024 merged commit e146e91 into dev Mar 2, 2026
6 checks passed
@divyanshub024 divyanshub024 deleted the dv/readme branch March 2, 2026 09:12
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.

2 participants