Skip to content

Change extension category #749

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
jreineckearm merged 2 commits into from
Jan 21, 2026
Merged

Change extension category #749

jreineckearm merged 2 commits into from
Jan 21, 2026

Conversation

@jreineckearm
Copy link
Collaborator

@jreineckearm jreineckearm commented Jan 20, 2026 •
edited
Loading

Fixes

Changes

  • Removes "Extension Packs" category from pack to render separate Extension Pack tab.
  • Use .png QLTY badges instead of .svg ones. The latter seems to be no longer allowed by vsce. Only showed in local builds because CI strips the badges before packaging.

Screenshots

image image

Checklist

  • 🤖 This change is covered by unit tests (if applicable).
  • 🤹 Manual testing has been performed (if necessary).
  • 🛡️ Security impacts have been considered (if relevant).
  • 📖 Documentation updates are complete (if required).
  • 🧠 Third-party dependencies and TPIP updated (if required).

@jreineckearm
Remove 'Extension Packs' category
38cab67 
to change its rendering in VS Code Extension Manager.

Signed-off-by: Jens Reinecke <[email protected]>
@jreineckearm
Use .png variants of QLTY badges
883fd20 
Signed-off-by: Jens Reinecke <[email protected]>
@qltysh
Copy link

qltysh bot commented Jan 20, 2026

Qlty

Coverage Impact

This PR will not change total coverage.

🚦 See full report on Qlty Cloud »

🛟 Help

  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@jreineckearm jreineckearm mentioned this pull request Jan 20, 2026
Open
jkrech
jkrech approved these changes Jan 20, 2026
View reviewed changes
Copy link
Member

@jkrech jkrech left a comment

Choose a reason for hiding this comment

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

LGTM

soumeh01
soumeh01 approved these changes Jan 20, 2026
View reviewed changes
Copy link
Collaborator

@soumeh01 soumeh01 left a comment

Choose a reason for hiding this comment

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

LGTM

@jreineckearm jreineckearm merged commit 4f0b978 into main Jan 21, 2026
22 checks passed
@jreineckearm jreineckearm deleted the change-extension-category branch January 21, 2026 05:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jkrech jkrech jkrech approved these changes

@soumeh01 soumeh01 soumeh01 approved these changes

@omarArm omarArm Awaiting requested review from omarArm

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jreineckearm @jkrech @soumeh01