Skip to content

docs: Refine documentation for clarity and consistency#2038

Merged
imaNNeo merged 1 commit intoimaNNeo:mainfrom
AminMemariani:main
Apr 15, 2026
Merged

docs: Refine documentation for clarity and consistency#2038
imaNNeo merged 1 commit intoimaNNeo:mainfrom
AminMemariani:main

Conversation

@AminMemariani
Copy link
Copy Markdown
Contributor

  • Improved grammar and clarity in CONTRIBUTING.md, README.md, and various documentation files.
  • Standardized phrasing for properties in chart data classes across documentation.
  • Enhanced descriptions for touch handling and tooltip functionalities in scatter, pie, and radar charts.
  • Corrected minor typographical errors and formatting issues throughout the documentation.

@imaNNeo
Copy link
Copy Markdown
Owner

imaNNeo commented Dec 20, 2025

Thank you, it looks good to me.
But now there are some conflicts. (Maybe because of your changes in other branches).

So please update your branch, then we can merge it!

@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 21, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.00%. Comparing base (9f74754) to head (6042278).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2038   +/-   ##
=======================================
  Coverage   93.00%   93.00%           
=======================================
  Files          50       50           
  Lines        3917     3917           
=======================================
  Hits         3643     3643           
  Misses        274      274           
Flag Coverage Δ
flutter 93.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@AminMemariani
Copy link
Copy Markdown
Contributor Author

Thank you, it looks good to me. But now there are some conflicts. (Maybe because of your changes in other branches).

So please update your branch, then we can merge it!

You are welcome. Although the branch was updated, there were pipeline constraints that were blocking my commits, so I had to fix them.

@imaNNeo
Copy link
Copy Markdown
Owner

imaNNeo commented Feb 25, 2026

Oh sorry, I made some changes in #2057
So it seems there are some conflicts now.

- Improved grammar and clarity in CONTRIBUTING.md, README.md, and various documentation files.
- Standardized phrasing for properties in chart data classes across documentation.
- Enhanced descriptions for touch handling and tooltip functionalities in scatter, pie, and radar charts.
- Corrected minor typographical errors and formatting issues throughout the documentation.
@AminMemariani AminMemariani changed the title Refine documentation for clarity and consistency docs: Refine documentation for clarity and consistency Mar 16, 2026
@imaNNeo
Copy link
Copy Markdown
Owner

imaNNeo commented Apr 10, 2026

I see some changes in our unit tests.
Can you please revert them and open a new PR for them to keep this PR's scope only for the documentation?

@AminMemariani
Copy link
Copy Markdown
Contributor Author

I see some changes in our unit tests.
Can you please revert them and open a new PR for them to keep this PR's scope only for the documentation?

The 9 test commits are dropped from the branch. they're still reachable via reflog locally (HEAD was 6042278)

@imaNNeo imaNNeo merged commit 1d53ff9 into imaNNeo:main Apr 15, 2026
5 checks passed
@imaNNeo
Copy link
Copy Markdown
Owner

imaNNeo commented Apr 15, 2026

Thanks for your contribution!

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