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.

Sign up for GitHub

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

Jump to bottom

Code Cleanup #36

Merged
merged 2 commits into from Sep 25, 2016
Merged

Code Cleanup #36

merged 2 commits into from Sep 25, 2016

Conversation

ffflorian
Copy link
Collaborator

@ffflorian ffflorian commented Sep 25, 2016 •
edited

  • Replaced tabs with spaces
  • Unified if .. then
  • Unified indentation
  • Fixed a few typos

No functionality was changed.

@ffflorian
Cleanup
ea41ab3 
+ Replaced tabs with spaces
+ Unified if .. then
+ Fixed typo in 'DOCTYPE'
+ Unified indentation
@ffflorian
Cleanup: More indentation fixes
b85d0da
@ffflorian ffflorian changed the title Cleanup Code Cleanup Sep 25, 2016
@enoch85
Copy link
Member

enoch85 commented Sep 25, 2016

@ffflorian THANKS ALOT! I've been thinking of doing this for some time now, but never had the time. Welcome to the Nextcloud VM repo. 👍 💯 💯

@enoch85 enoch85 merged commit b2f1a6b into nextcloud:master Sep 25, 2016
@enoch85 enoch85 mentioned this pull request Sep 25, 2016
Closed
1 task
@enoch85
Copy link
Member

enoch85 commented Sep 25, 2016

@ffflorian If you have some time left over, please help me here as well: https://github.com/enoch85/ownCloud-VM

Thanks!

@ffflorian
Copy link
Collaborator Author

If you have some time left over, please help me here as well: [...]

Will do :)

@WaaromZoMoeilijk
Copy link
Member

Will give it a test run tonight or tomorrow! Good job!

WaaromZoMoeilijk pushed a commit that referenced this pull request Feb 6, 2017
Merge pull request #36 from nextcloud/master
3c621e9 
Passman fixes (#118)
@flostyen flostyen mentioned this pull request Apr 4, 2018
Closed
@sirmauri2 sirmauri2 mentioned this pull request May 14, 2019
Closed
@RKlarmann RKlarmann mentioned this pull request Apr 8, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ffflorian @enoch85 @WaaromZoMoeilijk