Skip to content

Remove UTF-8 BOM if necessary - #155

Merged
madtibo merged 1 commit into
dalibo:masterfrom
maresb:patch-1
May 26, 2021
Merged

Remove UTF-8 BOM if necessary#155
madtibo merged 1 commit into
dalibo:masterfrom
maresb:patch-1

Conversation

@maresb

@maresb maresb commented May 12, 2021

Copy link
Copy Markdown
Contributor

Closes #154

@madtibo madtibo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems there is an indentation problem.
The indentation is sadly not consistent throughout the script. I usually use the one form the surrounding block. Can you do the same?
Thanks in advance

@maresb

maresb commented May 26, 2021

Copy link
Copy Markdown
Contributor Author

@madtibo good catch, thanks. (I'm used to my editor converting tabs into spaces, but was using the GitHub editor in this case.)
I have force-pushed the fix.

@madtibo madtibo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks

@madtibo
madtibo merged commit 2aaecc8 into dalibo:master May 26, 2021
@maresb
maresb deleted the patch-1 branch May 26, 2021 16:18
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.

Opening USE [master] line not understood

2 participants