Skip to content

docs: Delete Conslusions section from concepts - #1068

Merged
szaganek merged 2 commits into
masterfrom
docs-delete-conclusions-from-concepts
Jul 29, 2026
Merged

docs: Delete Conslusions section from concepts#1068
szaganek merged 2 commits into
masterfrom
docs-delete-conclusions-from-concepts

Conversation

@szaganek

@szaganek szaganek commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Deletes Conclusions section from all topics under Concepts. Applies to next and latest doc versions.

@szaganek
szaganek requested a review from vdusek July 29, 2026 06:57
@szaganek szaganek self-assigned this Jul 29, 2026
@szaganek szaganek added documentation Improvements or additions to documentation. t-docs Issues owned by technical writing team. labels Jul 29, 2026
@codecov

codecov Bot commented Jul 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.89%. Comparing base (67445ba) to head (4a59c33).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1068      +/-   ##
==========================================
+ Coverage   91.83%   91.89%   +0.06%     
==========================================
  Files          51       51              
  Lines        3232     3232              
==========================================
+ Hits         2968     2970       +2     
+ Misses        264      262       -2     
Flag Coverage Δ
e2e 35.33% <ø> (ø)
integration 57.08% <ø> (ø)
unit 83.26% <ø> (+0.06%) ⬆️

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

☔ View full report in Codecov by Harness.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vdusek vdusek left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Did you check that we are not losing any valuable information and that the conclusion contained only summaries/recaps?

Second, there is no need to duplicate this for the versioned v4 docs, we have a CI pipeline that handles the copying. Please remove the duplication.

@szaganek

Copy link
Copy Markdown
Contributor Author

@vdusek, I did check what I'm deleting from docs, no worries. Conclusions didn't carry any valuable information.

@vdusek vdusek left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@szaganek
szaganek merged commit 5c3a240 into master Jul 29, 2026
31 checks passed
@szaganek
szaganek deleted the docs-delete-conclusions-from-concepts branch July 29, 2026 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation. t-docs Issues owned by technical writing team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants