Skip to content
This repository has been archived by the owner on Nov 2, 2018. It is now read-only.

Rebase of #29 #56

Merged
merged 2 commits into from
Aug 11, 2015
Merged

Rebase of #29 #56

merged 2 commits into from
Aug 11, 2015

Conversation

Henni
Copy link
Contributor

@Henni Henni commented Aug 11, 2015

fixes #29

@jancborchardt
Copy link
Contributor

Please review @brantje @mozzwald @DJaeger @v1r0x :)

@DJaeger
Copy link
Contributor

DJaeger commented Aug 11, 2015

Why is the diff so curios?
Seems to be ok.

@LukasReschke
Copy link

Because he fixed indentation (or something like that). See https://github.com/owncloud/maps/pull/56/files?w=1 for a non-whitespace diff.

@Henni
Copy link
Contributor Author

Henni commented Aug 11, 2015

@DJaeger There was a mix of unix and windows line-endings. I converted them to unix line endings.

@DJaeger
Copy link
Contributor

DJaeger commented Aug 11, 2015

This parameter was new to me.
Everywhere is something to learn. ;-)

With this view I can say: 👍 :-)

jancborchardt added a commit that referenced this pull request Aug 11, 2015
@jancborchardt jancborchardt merged commit a82fdfb into master Aug 11, 2015
@jancborchardt jancborchardt deleted the fix-hash branch August 11, 2015 18:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants