-
Notifications
You must be signed in to change notification settings - Fork 26
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.
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
Publish to the VS Code marketplace #119
Comments
See also #40 |
@jonahgraham , there is some interest on our side to get this published to the VS Code marketplace as a CDT Cloud extension. We would be happy to pick this up. Any objections or thoughts on this? |
No objection - quite the opposite. As mentioned in the last cdt cloud call, the build infra here is quite sorely out of date, this is what I summarized at the call, but I don't know when I will have time to do this:
|
I'm hoping it should be quite easy to copy the CI from one of our other extensions. |
Great, thanks, @jonahgraham and @thegecko !
Yes, my intention is to start from existing extensions like the "Inspector" extensions. Unless you have something else in mind. We can then discuss on the PR if this is sufficient or should be further refined/provide more functionality. I might just need some assistance with setting up secrets as I certainly don't have the required rights to do so myself. Also happy to have a look at dependencies (I see dependabot was busy here already). But probably as a separate PR. And would need some support from others to test their use cases. |
Sounds good to me. I have nothing else in mind.
I am hoping @thegecko can handle this part, but lmk if you need me for any of it. |
We have org wide secrets set up for publishing all extensions, copying another CI workflow should 'just work'. |
Nice! |
Should be closed in #130 |
The good news is that the |
Should be fixed with node 20 update in #131. |
This doesn't seem to have been published to the main VS Code marketplace, any reason why it shouldn't be?
The text was updated successfully, but these errors were encountered: