Skip to content

Conversation

@thrzl
Copy link

@thrzl thrzl commented Feb 22, 2022

adds a simple hello world example

@descovi
Copy link

descovi commented Feb 24, 2022

Maybe you add some complexity with the creation of a file 🤔

@jasonnoble
Copy link

If we follow the Style Guide is this PR more of a LGTM or a CIAL situation?

Does the Style Guide need updating to handle adding blank lines?

@thrzl
Copy link
Author

thrzl commented Feb 26, 2022

Considering the style guide, the filesize should be zero. Even adding nothing () will raise the size to approx 4 bytes, failing the du check. I have checked this code for performance issues and checked that this is style-conforming. If you do find any issues, do tell me.

@thrzl
Copy link
Author

thrzl commented Feb 26, 2022

Maybe you add some complexity with the creation of a file 🤔

I could always make the filename:

or ‎.no

@thrzl thrzl closed this by deleting the head repository Apr 23, 2023
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.

5 participants