-
Notifications
You must be signed in to change notification settings - Fork 278
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
Badly documented and meaningless variable names #7
Comments
You should watch the tutorial lol |
Well i have watched the tutorial, |
yes i implemented my code step by step very helpful
…On Tue, 1 Jun 2021 at 13:31, Shikhar Sharma ***@***.***> wrote:
Well i have watched the tutorial,
I don't know if you have just watched the tutorial or tried to implement
the code yourself,
But if you do try to write each line of code yourself and get it working,
i think you will get what i mean.
Just to give some reference using todo,todos,todolist and similar
variables at the same time is not reader friendly
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#7 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AMXTCNC4XUN2HXAWOPAOPFTTQSLELANCNFSM4V24XLEQ>
.
|
ok i try to explain
…On Tue, 1 Jun 2021 at 18:32, Mirza Zaid Hafeez ***@***.***> wrote:
yes i implemented my code step by step very helpful
On Tue, 1 Jun 2021 at 13:31, Shikhar Sharma ***@***.***>
wrote:
> Well i have watched the tutorial,
> I don't know if you have just watched the tutorial or tried to implement
> the code yourself,
> But if you do try to write each line of code yourself and get it working,
> i think you will get what i mean.
> Just to give some reference using todo,todos,todolist and similar
> variables at the same time is not reader friendly
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <#7 (comment)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AMXTCNC4XUN2HXAWOPAOPFTTQSLELANCNFSM4V24XLEQ>
> .
>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The code in the repo is badly written and is not reader friendly,
There is no documentation or comments what each function does, also variable names chosen don't convey a meaning.
In short code is not reader friendly.
The text was updated successfully, but these errors were encountered: