-
Notifications
You must be signed in to change notification settings - Fork 5
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
feature/typescript tutorial #1
base: master
Are you sure you want to change the base?
Conversation
@@ -0,0 +1,469 @@ | |||
# Clear code |
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.
This is same as we have in javascript
file
|
||
Now I am creating some simple form from login to site. | ||
|
||
![Form typing](../../assets/clear-code/form-type.png) |
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.
It's better to write code instead of using pictures.
|
||
![Input fields](../../assets/clear-code/input-field-refactoring.png) | ||
|
||
Reading got easier. The name of the component corresponds to the problem it solves. The purpose of the element is evident. There is less code. So we are heading in the right direction! |
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.
Decrease amount of words in this file
|
||
[Return to Typescript](./README.md). | ||
|
||
#### `Partial <T>` - constructs a type with all properties of `T` set to optional. |
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.
Decrease explanation here and put this info to all typescript document
4. [Development](docs/development.md) | ||
5. [Tools](docs/tools.md) | ||
6. [Git Flow](docs/git.md) | ||
7. [Clear code](docs/clear-code/README.md) |
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.
Better put this info about patterns to react doc with namereact-clean-code
After reading these articles, not merge, please, I combine commits.