Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

[WIP] Stage lines from editor - #93

Closed
kuychaco wants to merge 8 commits into
masterfrom
ku-stage-highlighted-lines
Closed

kuychaco wants to merge 8 commits into
masterfrom
ku-stage-highlighted-lines

Conversation

@kuychaco

Copy link
Copy Markdown
Contributor

TODO:

  • add specs
  • refactor staging logic into FileDiff
  • implement unstaging

Closes #46

Signed-off-by: Michelle Tilley <binarymuse@github.com>
@joshaber

Copy link
Copy Markdown
Contributor

Oh, FYI in case it wasn't clear: CI doesn't pass right now because of an annoying node-gyp bug 😢 So I just run it locally and then weep in a corner.

Comment thread lib/common.js Outdated
return objects
}

export function any <T> (arr: Array<T>, predicate: (item: T, idx: number, arr: Array<T>) => boolean): boolean {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could we use Array.prototype.some instead?

@joshaber

Copy link
Copy Markdown
Contributor

😻

Comment thread lib/git-package.js Outdated
.map(hunk => hunk.getLines())
.reduce((acc, lines) => acc.concat(lines), [])
.filter(line => line.isChanged() && anyRangesIntersectLine(line))
.forEach(line => line.stage())

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

FYI, #101 changed how staging works. You'll now need to call FileListViewModel.stageLines (and no more transacting!).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

JK, now it's GitStore.stageLines

Signed-off-by: Michelle Tilley <binarymuse@github.com>
Signed-off-by: Michelle Tilley <binarymuse@github.com>
@kuychaco
kuychaco force-pushed the ku-stage-highlighted-lines branch from df7b78f to 26fcded Compare March 29, 2016 01:23
Signed-off-by: Michelle Tilley <binarymuse@github.com>
Signed-off-by: Michelle Tilley <binarymuse@github.com>
Signed-off-by: Michelle Tilley <binarymuse@github.com>
Signed-off-by: Michelle Tilley <binarymuse@github.com>
@iolsen iolsen added this to the 3: Next Public Relase milestone Jun 1, 2016
@kuychaco

Copy link
Copy Markdown
Contributor Author

Put on hold to prioritize github-package features

@BinaryMuse

Copy link
Copy Markdown
Contributor

This is so absolutely and completely out of date that I'mma just 🔥 it

@BinaryMuse BinaryMuse closed this Jan 7, 2017
@smashwilson
smashwilson deleted the ku-stage-highlighted-lines branch September 21, 2018 00:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants