Skip to content

Add contextual-theme#5622

Open
Amarin38 wants to merge 5 commits intozed-industries:mainfrom
Amarin38:add-contextual-zed-theme
Open

Add contextual-theme#5622
Amarin38 wants to merge 5 commits intozed-industries:mainfrom
Amarin38:add-contextual-zed-theme

Conversation

@Amarin38
Copy link
Copy Markdown

No description provided.

@cla-bot
Copy link
Copy Markdown

cla-bot bot commented Apr 15, 2026

We require contributors to sign our Contributor License Agreement, and we don't have @Amarin38 on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@Amarin38
Copy link
Copy Markdown
Author

@cla-bot check

@cla-bot cla-bot bot added the cla-signed label Apr 15, 2026
@cla-bot
Copy link
Copy Markdown

cla-bot bot commented Apr 15, 2026

The cla-bot has been summoned, and re-checked this pull request!

Copy link
Copy Markdown
Member

@MrSubidubi MrSubidubi left a comment

Choose a reason for hiding this comment

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

Please make sure to follow the extension publishing guide in detail in order to have this published. You are missing a few steps, please update your PR accordingly.

Also, please make sure your extension fulfills the extension publishing prerequisites. Once you did the necessary updates, please update everything related here. Thanks!

@MrSubidubi MrSubidubi marked this pull request as draft April 15, 2026 17:44
@Amarin38 Amarin38 marked this pull request as ready for review April 15, 2026 23:53
@Amarin38
Copy link
Copy Markdown
Author

Hi @MrSubidubi ! I have updated the extension name/id to comply with the rules, and registered it correctly in extensions.toml using pnpm sort-extensions.

Comment thread extensions/contextual-zed
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You'll still need to fix the path here

Comment thread .gitmodules
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

And here

Comment thread extensions.toml
version = "2.0.1"

[contextual-themes]
submodule = "extensions/contextual-themes"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

As this submodule does not exist currently (you need to rename the submodule to the name chosen here)

@MrSubidubi MrSubidubi changed the title Add Contextual-Zed: A specialized theme pack for Java, C++ and Python Add contextual-theme Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants