Skip to content

Conversation

@roycewilliams
Copy link
Contributor

@roycewilliams roycewilliams commented Jul 14, 2019

Note that the whitespace changes in sha256.md are due to converting from DOS to Unix line separators to match the other files. And actually credit to @hops for the kernel work as well. 😄

@spaze spaze merged commit 746aa97 into spaze:master Jul 14, 2019
@spaze
Copy link
Owner

spaze commented Jul 14, 2019

Hey, awesome work, thanks! I took the liberty and modified your pull request. I've converted the newlines in my repo (thanks for spotting those CRLFs) so this PR didn't need to change them.

@roycewilliams
Copy link
Contributor Author

All credit to @Chick3nman and @hops for the work - I just supplied some horsepower. And I totally trust your judgment on modifying PRs as needed. :D

@spaze
Copy link
Owner

spaze commented Jul 14, 2019

FYI, one more change, the backslash is not part of the string, just to escape the backtick (it wasn't a magic hash otherwise):

- CGq'v]\`1
+ CGq'v]`1

@roycewilliams
Copy link
Contributor Author

Yikes - good catch, sorry I missed that!

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.

2 participants