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

fix inconsistent indentation in README #111

Merged
merged 2 commits into from
Jun 26, 2019
Merged

Conversation

aspiers
Copy link
Contributor

@aspiers aspiers commented Jun 2, 2019

It was a mix of tabs and spaces, and the tabs were obviously meant to be interpreted as being 4 columns wide, which is a much less common interpretation than 8 columns. So make the file self-consistent and unambiguous by replacing each tab with 4 spaces.

Then fix a couple of issues with indentation in the example docker-compose file.

It was a mix of tabs and spaces, and the tabs were obviously meant to
be interpreted as being 4 columns wide, which is a much less common
interpretation than 8 columns.  So make the file self-consistent and
unambiguous by replacing each tab with 4 spaces.

Then fix a couple of issues with indentation in the example
docker-compose file.
@luizeof luizeof self-assigned this Jun 7, 2019
README.md Outdated Show resolved Hide resolved
Accept some fixes from @shawncarr.

Co-Authored-By: Shawn Carr <[email protected]>
@aspiers
Copy link
Contributor Author

aspiers commented Jun 26, 2019

Please could this be merged soon? We are already seeing duplicate efforts crop up, e.g. #113. Thanks!

@luizeof
Copy link
Member

luizeof commented Jun 26, 2019

thanks @aspiers

I always dedicate my Friday morning to this project ;)

@luizeof luizeof merged commit 8ff6cfa into mautic:master Jun 26, 2019
@aspiers aspiers deleted the remove-tabs branch June 27, 2019 11:13
@aspiers
Copy link
Contributor Author

aspiers commented Jun 27, 2019

Thanks a lot @luizeof!

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

Successfully merging this pull request may close these issues.

3 participants