-
Notifications
You must be signed in to change notification settings - Fork 31
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
Class basic syntax #40
Conversation
Solved all suggestion /done |
@Benyaak, you're not a romanian native speaker, am i right ? :) |
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.
Overall, good job !
Please make the requested changes. After it, add a comment "/done". |
|
You can't merge because of conflicts. Can you fix the conflicts (know what it is?) |
Yep but GitHub wouldn't let me edit the files. I think this person unticked the "let maintainers edit code" box |
Maybe check it out locally, merge/edit and then close this PR? |
I think he may have deleted his Not sure exactly, I can copy/paste his code and modify it in a new Is there another way? |
|
me@os:~/Documents/GitHub/ro.javascript.info$ git fetch -fu origin refs/pull/40/head:pr/40
fatal: couldn't find remote ref refs/pull/40/head
|
From this repo:
|
I made a new branch |
See this PR #146 |
thanks a lot 🙏️ |
Finished the class basic syntax lesson, I hope everything is okay. Waiting for a review/translation suggestions. Thanks a lot.