[MonoDevelop] Git add-in
"Andrés G. Aragoneses"
knocte at gmail.com
Wed Jul 28 19:04:47 EDT 2010
Agreed in general with all opinions from Lluis. Just a couple of
comments inline:
El 28/07/10 20:07, Lluis Sanchez Gual escribió:
> El dc 28 de 07 de 2010 a les 10:41 -0400, en/na Dale Ragan va escriure:
>>> So the Git add-in can be used pretty much like the Subversion add-in.
>> The only big difference is that changes have to be explicitly pushed to
>> the remote repository using the Push command. Feedback about how to
>> improve this workflow is welcome.
>>>
>> I see you are finding the issues that I was faced with when I started on
>> this venture. I'm glad you're back, so we can work on this together.
>> Here are some ideas that I came up with while thinking about this feature
>> over the last month:
>>
>> Since we will be shipping two version control systems out of the box now,
>> I think it would be beneficial to set a setting on which Version Control
>> System MonoDevelop should use by default.
>
> This is not necessary because MD already detects which project is using
> which version control system. Options shown in the context menu can be
> specific to the version control of the project.
>
> If you are concerned about the Checkout command, which has a special
> meaning in git, we can change the name and use something more explicit
> like "Get Project from Repository" or something like that.
>
>> Then when opening existing
>> projects we can scan for the system being used and set this value. This
>> will simplify and allow segregation of the commands and not present
>> options that are not valid based on the system being used. This will also
>> allow us to add configuration settings for the Version Control System that
>> the user can set. For instance, one feature I had in mind, which looks
>> like you implemented an early version of, is to display the current branch
>> beside the solution name. A configuration setting could actually
>> customize the output of that string. This would also be a good place to
>> set globally the files to ignore automatically when creating a new
>> repository.
>
> This can be done without having to set a default VC system. Specific VCS
> can extend the solution pad and customize it however they want, just
> like I did with the branch label in the solution node.
>
>>
>> The commands available in the context menu should be in sync with the
>> command line options. This helps with uniformity and makes going back and
>> forth a lot easier. It will also enhance the workflow by presenting
>> commands in a way that are already familiar with the command line and vice
>> versa.
>
> I disagree with that point of view. The goal of an IDE is to simplify
> the development process. If we replicate every single git command, we
> are not simplifying, we are exposing all the complexity of git.
>
> As a user I don't want to have to deal with the complexity of git every
> day. There are only a few commands I need to use in everyday work
> (review changes, commit changes, update from repo). I want MD to take
> the correct decisions for me in order to perform those operations with
> git. In case I need to do more advanced git operations, I can always
> open a console and use the git command.
>
>>
>> Here's a list of commands that I think should be implemented:
>>
>> File Menu
>> Initialize - Create a new git repository
>
> We can create a new "Create repository" command, which can also be
> implemented for SVN.
>
>> Clone - Clone an existing git repository
>
> It is the Checkout command, which can be renamed to "Get Project from
> Repository", shared with SVN.
>
>>
>> Project Context Menu
>> Commit - Commit changed files to local repository allowing the user to
>> check which changes to stage for the commit
>
> This is the current Review Changes view and the Commit command.
>
>> Push - Push changes to remote repository
>
> This command is new and specific to Git. Ok to add it.
You could also create a new generic DistributedVersionControl class
outside of the Git addin that adds a Push abstract operation, so other
addins that inherit from it can define their own Push behavior.
>> Pull - Pull changes from remote repository
>> Fetch - Fetch changes from remote repository
>
> Pull and Fetch are implemented by the "Update" command, so there is no
> need to have them as separate commands.
>
>> Create Branch - Create a new branch
>> Checkout Branch - Checkout an existing branch
>> Merge - Merge changes from one branch to another
>
> We can have a branch manager for doing those operations.
>
>> Create Tag - Create a new Tag
>
> We can add this command.
>
>> Log - View commit log
>
> That's the existing Log command.
>
>> Reset HEAD - Revert changes to last commit
>
> There is a Revert command in the SVN add-in which can be generalized.
>
>> Stash - Stash changes and reverting to last commit
>> UnStash - Unstash a stash object
>
> I'm not sure we need those. MD can automatically use stash for the most
> common operations, such as when pulling or when switching branches.
>
>> Add - Add changes files to staging
>
> I don't think the concept of "stagging" is really useful in the IDE.
> Stagging is useful in the command line because it allows you to select
> what do you want to commit. But the IDE already has a GUI for selecting
> what to commit (the review changes view), so this command is not
> necessary.
>
>> Ignore - Add file to .gitignore
>> Apply Patch - Apply a patch to current branch
>
> We don't have those commands, but it might be useful to have them for
> both SVN and Git.
>
>> Create Patch - Create a patch based on last commit
>
> Already exists.
For this one, it would be useful to place it in another place because
now with DVCS there's another use case: you commit, and then you want to
create the patch of an already-committed-locally changeset so other
person can review it before pushing it.
>>
>> A pad for the history view showing the commits for the current branch.
>> This feature will present the commits in a nice and readable format. The
>> user will then be allowed to view changes for a commit and compare.
>
> This is the Log command. This command, when executed on a directory,
> used to show the commit history, but this seems to be broken in trunk.
>
>>
>> A pad for managing Git repositories. This feature will present the .git
>> directory in an easy to read and presentable way allowing management of
>> remotes and branches for instance.
>
> Ok, although it would be a dialog, not a pad.
>
>>
>> Of course this isn't everything probably, but just a short list of
>> features that I planned on doing. Any other thoughts?
>
> That's all!
> Lluis.
My 2 cents!
Andres
--
More information about the Monodevelop-list
mailing list