sublimehq / sublime_merge

Issue tracker for Sublime Merge
https://www.sublimemerge.com
276 stars 14 forks source link

API Use Case Tracker #1500

Open TerminalFi opened 2 years ago

TerminalFi commented 2 years ago

Problem description

Sublime Merge is a powerful tool already but would greatly benefit from a similar API interface like its relative Sublime Text. This issue is going to act as a tracker for features which could be possible with an API interface.

API Ideas

ratijas commented 2 years ago

GitHub & GitLab reviews: integrate with Pull Request / Merge Request review workflow. Display review comments inline with the code, and let users reply (with respective Markdown flavor preview) and resolve threads.

Also, add an action to open PR/MR web page for a branch.

TerminalFi commented 2 years ago
srbs commented 2 years ago

for tracking purposes: see also: #267, #394, #964, #1101

themilkman commented 2 years ago

I always wanted to have a periodic check whether there are upstream changes (aka forgot to pull). For not to get in troubles e.g. with git push --force-with-lease I'd check the output of git fetch --dry-run. That'd be something I could imagine to make with a plugin API.

srbs commented 2 years ago

@themilkman, I'm curious on what kind of trouble can you get in with git fetch that would necessitate --dry-run?

themilkman commented 2 years ago

@themilkman, I'm curious on what kind of trouble can you get in with git fetch that would necessitate --dry-run?

@srbs You may correct me, but iirc "normal" a git fetch in the background by the plugin just before I do a git push --force-with-lease would override the HEAD on origin because git thinks "yeah, it's already here, you know what you do". A --dry-run afaik doesn't update the local refs of the remotes. Thus, you get the information "hey, someone pushed smth" for the plugin but wouldn't update the actual local repo with the refs. Which should prevent me from force-pushing my commit over another which I didn't know about yet because it was automatically fetched by the plugin before I could notice it.

srbs commented 2 years ago

@themilkman, ah yeah, makes sense. I keep forgetting exactly how --force-with-lease works.

themilkman commented 2 years ago

Also, one of the first things I'd try to port is https://github.com/evandrocoan/RememberCommandPaletteInput ...

themilkman commented 1 year ago

What I also could imagine would be an visual indicator in the 'Branches' list on the entries if they are already merged into the currently checked out one.