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

added EOF #12

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@Stefany93

Stefany93 commented May 26, 2015

Having an EOF line makes it easier for the developer to see where the file ends.

@audreyt

This comment has been minimized.

Show comment
Hide comment
@audreyt

audreyt May 26, 2015

Owner

Could this be __END__ instead?

Owner

audreyt commented May 26, 2015

Could this be __END__ instead?

@Stefany93

This comment has been minimized.

Show comment
Hide comment
@Stefany93

Stefany93 May 26, 2015

Sure, both EOF and END denote the end of the file in a very clear way.

Stefany93 commented May 26, 2015

Sure, both EOF and END denote the end of the file in a very clear way.

@audreyt audreyt closed this in 8093a81 May 26, 2015

@audreyt

This comment has been minimized.

Show comment
Hide comment
@audreyt

audreyt May 26, 2015

Owner

Done! Thanks for the suggestions 🌈

Owner

audreyt commented May 26, 2015

Done! Thanks for the suggestions 🌈

@Stefany93

This comment has been minimized.

Show comment
Hide comment
@Stefany93

Stefany93 May 26, 2015

No worries, thanks for accepting.

Stefany93 commented May 26, 2015

No worries, thanks for accepting.

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