Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve README punctuation. #45

Merged
merged 2 commits into from Mar 8, 2017
Merged

Improve README punctuation. #45

merged 2 commits into from Mar 8, 2017

Conversation

m4z
Copy link
Contributor

@m4z m4z commented Mar 2, 2017

(And fix a few typos.)

@cleech
Copy link
Collaborator

cleech commented Mar 7, 2017

Looks like this now conflicts with your earlier changes, want to resolve it?

(for future reference you can have multiple change commits on one branch and send then in a single pull request when they're all related like this)

@m4z
Copy link
Contributor Author

m4z commented Mar 8, 2017

Yeah, seems like this noob right here got lost in his own branches. Should be fixed now.

I still have some potentially controversial changes coming up though, so I deemed it better to split the different kinds of changes. Will see if I can keep the branches to a minimum. 😉

@cleech cleech merged commit b9604c4 into open-iscsi:master Mar 8, 2017
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.

None yet

2 participants