Skip to content

Comments

Fix Formatting of Table#3401

Draft
tirth-patel-nc wants to merge 2 commits intomasterfrom
tirth-patel-nc-patch-2
Draft

Fix Formatting of Table#3401
tirth-patel-nc wants to merge 2 commits intomasterfrom
tirth-patel-nc-patch-2

Conversation

@tirth-patel-nc
Copy link
Contributor

Fix Formatting of Table

Before After
Screenshot 2026-02-20 at 6 31 07 PM Screenshot 2026-02-20 at 6 33 40 PM

Fix Formatting of Table
Fix formatting for installed SDK packages section.
@tirth-patel-nc
Copy link
Contributor Author

Screenshot 2026-02-20 at 6 34 14 PM

check https://docs.codemagic.io/specs-linux/ubuntu-24.04/ for fixed version.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes markdown formatting issues in the Windows machine specifications document. The changes correct the placement of a code fence and remove extraneous blank lines that were breaking table rendering in the Android SDK packages section.

Changes:

  • Moved closing code fence from line 28 to line 36 (after command execution output)
  • Removed blank lines between table rows in the SDK packages table to ensure proper markdown rendering

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dtrdic
Copy link
Contributor

dtrdic commented Feb 20, 2026

This one also seems outside his place
Screenshot 2026-02-20 at 14 14 23

But wait with this one as well.. I believe the devs have a different workflow which takes care of the updates for the machine specifications.. don't know if it takes care of the layout as well, so i'll add @helinanever here as well to comment

@tirth-patel-nc tirth-patel-nc marked this pull request as draft February 20, 2026 14:04
@tirth-patel-nc
Copy link
Contributor Author

collapsible thing won't be an issue on the actual page.

Copy link
Contributor

@dtrdic dtrdic left a comment

Choose a reason for hiding this comment

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

LGTM

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