-
Notifications
You must be signed in to change notification settings - Fork 88
Update README.md #9
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
Conversation
this is my github contribution .feel free tog give constructive feedbacks
Congrats @meteora2002 on your first contribution. I suggest you to make README a bit more readable using correct formatting, currently, it does look rightly formatted. |
done |
@tuladhar please accept my pull requests |
Thanks @meteora2002 I've requested for some changes, please check. |
@tuladhar please check my pull request |
README.md
Outdated
.It is the exact opposite of json.loads. | ||
## THIS IS THE ENCODING DECODING LIST | ||
| JSON| Python | |
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.
Table is still not showing up correctly.
README.md
Outdated
-json.dumps() function converts a Python object into a json string | ||
.It is the exact opposite of json.loads. | ||
## THIS IS THE ENCODING DECODING LIST |
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.
Let's use normal text for it, and remove the heading formatting.
README.md
Outdated
- `Decoding is from JSON to python(final type)` | ||
## other commands |
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.
Let's rename it to "Encoding functions"
README.md
Outdated
`infile`-to check your Json file for syntax | ||
`outfile`-Write the output of the infile to the given outfile | ||
## note |
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.
Let's capitalize it.
done |
@tuladhar please check my pull request |
README.md
Outdated
THIS IS THE ENCODING DECODING LIST | ||
| JSON |Python | |
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.
Table is still not showing. You can learn about it here:
Also, @meteora2002 let's remove the "coming soon" label from the contents table for JSON and add link. |
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.
Thanks @meteora2002 for your contributions. Looks good to merge!
finished the json file in simple language
this is my github contribution .feel free to give constructive feedbacks