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
Primitives, Variables, Operators and Mutability #14
Conversation
@@ -23,5 +16,3 @@ fn main() { | |||
/* println(foo) */ // Also wouldn't work because custom str func not defined . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe function
instead func
.
@@ -30,7 +30,7 @@ If you don't want to use `\n` then you can use `println` instead. | |||
|
|||
## Comments | |||
|
|||
V supports single line comments `//` and mutli line comments `/* */`. They should be used for documenting the code for letting the other users know how the code works. It can also be used for temporarly commenting the code which has to be used later on. | |||
V supports single line comments `//` and mutli-line comments `/* */`. They should be used for documenting the code for letting the other users know how the code works. It can also be used for temporarily commenting the code which has to be used later on. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo on multi-line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
stay consistent with multi-line
. change single line
to single-line
.
@@ -0,0 +1,12 @@ | |||
## Keywords | |||
|
|||
V is a very small language so it has less keywords. There are around 25 keywords. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe change less
to fewer
maybe better
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
few
would sound better imo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ya... LGTM 👌
@Delta456 is it ready to merge? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@vbrazo It isn't |
@Delta456 i dont know... but some of my review is skipped... is up to you guys... i just review |
I will update the repo structure in one of next PRs.