-
Notifications
You must be signed in to change notification settings - Fork 30.9k
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
SCM - Add an actionButton
property to the SourceControl api
#133935
Comments
@lszomoru is there any plan for this to become available in the stable API? I'd like to use it in the DVC Extension for VS Code. Use case is here: iterative/vscode-dvc#1769. Some high level details about the extension can be found at microsoft/vscode-discussions#54. |
@mattseddon, thanks for the interest in the Action Button. The last two milestones we evolved the shape of the property in order to accommodate the "Commit" action button. I think that soon we will reach a phase where we can look at finalising the API. |
Any update on this becoming stable? |
@DavidGoldman, no progress was made in finalising the action button. Sorry. |
+1 to get this API |
@lszomoru if I'd like to go the hard way (e.g. copy/pasting some code from the |
@vasilev-alex, I do not believe that taking code from the |
@vasilev-alex if you are prepared to tolerate the constraints imposed on extensions that used proposed API (i.e. can't publish on Marketplace, and even after install from VSIX require a command-line argument to be specified each time or alternatively added to @lszomoru I sincerely hope this proposed API isn't going to follow the example of several others which have languished in proposed state for many years, often with no refinement occurring during that time, and giving IMO an unfair advantage to certain "blessed" extensions that are permitted to use them without having to overcome the constraints mentioned above. Are there specific reasons for not finalizing this one? |
Thinking out loud here, I wonder what the answer would be if someone with a subscription asked GitHub Copilot what it'll take to get a proposed API finalized. |
@gjsjohnmurray, thanks for the continued feedback. The proposal for the action button has evolved since its inception (ex: added description, and secondary actions). One aspect that we still have to decide on before finalization is what's the best approach for defining secondary action (ex: code vs. menu with static contributions). |
I would prefer secondary actions to be contributed by code. Having to predefine options in settings.json is a significant restriction the UX our Deltanji SCM extension can provide. Deltanji provides highly configurable workflow, so when we build our extension we cannot know which transfer routes a Deltanji site will want to offer to its devs. Of course, if our longstanding wish for dynamic (sub)menu support were granted then it would probably be fine for secondary actions to be implemented as a menu. |
actionButton
property to the SourceControl apiactionButton
property to the SourceControl api
Refs: #110882
This proposal adds a new
actionButton
property to theSourceControl
object, to allow SCM extensions to contribute a large "cta" style button to a repository.Here is the proposed api addition:
This is currenty being used by the built-in Git extension to provide the "Publish Changes" or "Sync Changes" button when there are unpublished or unpushed commits.
The text was updated successfully, but these errors were encountered: