Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Implement Stash API.#871

Closed
Therzok wants to merge 6 commits into
vNextfrom
therzok/stashApply
Closed

Implement Stash API.#871
Therzok wants to merge 6 commits into
vNextfrom
therzok/stashApply

Conversation

@Therzok
Copy link
Copy Markdown
Member

@Therzok Therzok commented Nov 14, 2014

Based on libgit2/libgit2#2705

Not in master yet, adding so I can test locally.

nulltoken
nulltoken reviewed Nov 14, 2014
View reviewed changes
Comment thread LibGit2Sharp/Core/NativeMethods.cs Outdated
Copy link
Copy Markdown
Member

@nulltoken nulltoken Nov 14, 2014

Choose a reason for hiding this comment

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

ZOMG! tabs! :trollface:

Copy link
Copy Markdown
Member Author

@Therzok Therzok Nov 14, 2014

Choose a reason for hiding this comment

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

Yeah, I know. I did it as fast as possible without caring what's going on. :P

Copy link
Copy Markdown
Member

@nulltoken nulltoken Nov 14, 2014

Choose a reason for hiding this comment

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

😉

@vbjay
Copy link
Copy Markdown

vbjay commented Dec 3, 2014

https://visualstudiogallery.msdn.microsoft.com/dbcb8670-889e-4a54-a226-a48a15e4cace --2013
https://visualstudiogallery.msdn.microsoft.com/3a96a4dc-ba9c-4589-92c5-640e07332afd --2012
https://visualstudiogallery.msdn.microsoft.com/d0d33361-18e2-46c0-8ff2-4adea1e34fef --2010

All have the feature to detect mixed tabs.

https://visualstudiogallery.msdn.microsoft.com/c8bccfe2-650c-4b42-bc5c-845e21f96328 This one saves your bacon by changing the settings to fit the design of which project you are on.

@Therzok
Copy link
Copy Markdown
Member Author

Therzok commented Dec 3, 2014

@vbjay I do my work in XS and it has auto-format on save, I just didn't do it in an IDE this time (vimmed my way through it since I was ssh'ing to my machine from college when this PR happened).

@vbjay
Copy link
Copy Markdown

vbjay commented Dec 3, 2014

Oh the fun with vim. :)

On Wed Dec 03 2014 at 11:33:17 AM Marius Ungureanu [email protected]
wrote:

@vbjay https://github.com/vbjay I do my work in XS and it has
auto-format on save, I just didn't do it in an IDE this time (vimmed my way
through it since I was ssh'ing to my machine from college when this PR
happened).

—
Reply to this email directly or view it on GitHub
#871 (comment).

@txdv
Copy link
Copy Markdown
Contributor

txdv commented Dec 4, 2014

Use "Show invisible chars"

@Therzok
Copy link
Copy Markdown
Member Author

Therzok commented Dec 13, 2014

Hey guys, I really mentioned "I wanted a quick patch" to test. I opened the PR on API discussion. I'll fix the PR up after the libgit2 PR is merged, since there's no worth in overworking if that PR is dropped or the API changes. 🐰

@nulltoken
Copy link
Copy Markdown
Member

nulltoken commented Dec 13, 2014

@Therzok spaces are a serious matter. That shouldn't be underestimated. Ever :trollface:

@dahlbyk dahlbyk mentioned this pull request Feb 1, 2015
Open
@Therzok Therzok force-pushed the therzok/stashApply branch from bebfe81 to 957530c Compare April 5, 2015 19:38
@Therzok
Copy link
Copy Markdown
Member Author

Therzok commented Apr 5, 2015

Okay, I'm advancing on this thing.

I got a few issues I have to point out.

  • This won't build on travis until we have that PR merged into upstream.
  • Stash apply libgit2#3018 (comment) - I'm testing this specific behavior.
  • Conflicts don't appear in the index for some reason. I'm an idiot who can't read docs x100.
  • Test coverage is horrible. 💃

@Therzok
Copy link
Copy Markdown
Member Author

Therzok commented Apr 5, 2015

I'm also experiencing some weird behaviour:

  • If I use ReinstateIndex, I get the new changes in the workdir.
  • Something screwed up (I updated GitCheckoutOpts, I probably missed something else?) I'm getting Invalid version 2049 on git_checkout_options).
  • No intermediary updates. :D

@Therzok Therzok force-pushed the therzok/stashApply branch from 3b8a9a7 to 0176b96 Compare April 6, 2015 15:21
@Therzok Therzok changed the title [DO NOT MERGE] Implement Stash API. [WIP] Implement Stash API. Apr 6, 2015
@Therzok Therzok changed the title [WIP] Implement Stash API. Implement Stash API. Apr 6, 2015
Therzok
Therzok reviewed Apr 6, 2015
View reviewed changes
Comment thread LibGit2Sharp/StashCollection.cs
Copy link
Copy Markdown
Member Author

@Therzok Therzok Apr 6, 2015

Choose a reason for hiding this comment

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

Probably this could be abstracted in a StashApplyOptions.

class StashApplyOptions
{
    StashApplyModifiers flags;
    CheckoutOptions checkoutOptions;
}

/cc @nulltoken

@Therzok Therzok mentioned this pull request Jun 3, 2015
Merged
@Therzok Therzok closed this Jun 3, 2015
@Therzok Therzok deleted the therzok/stashApply branch June 3, 2015 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Therzok @vbjay @txdv @nulltoken @dahlbyk