Skip to content

Here's a second version of the offline patch - #63

Open
Steve132 wants to merge 12 commits into
ymgve:masterfrom
Steve132:offline_patch-2
Open

Steve132 wants to merge 12 commits into
ymgve:masterfrom
Steve132:offline_patch-2

Conversation

@Steve132

@Steve132 Steve132 commented Jul 6, 2018

Copy link
Copy Markdown

I made a second version of the offline patch. This time it includes multiple commits from the refactor so you can see and verify each step. Then, it includes also a commit that is flagged that shows when the functionality was unchanged and it was just a simple refactor. Finally, it includes all the commits to add the functionality.

@Steve132 Steve132 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Literally this is only an indent. I don't know why git made it so weird.

@Steve132

Copy link
Copy Markdown
Author

Want any help merging this? The functionality is critical for a bunch of things and critical to my security model, and it can help other implementers too.

@Steve132

Steve132 commented Oct 9, 2018

Copy link
Copy Markdown
Author

Hey, I'll pay you a bounty to go through the head with me on hangouts and merge this (or a similar feature) into mainline. The way you want. Seriously.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant