Closed whirlxd closed 3 weeks ago
added this for hacktoberfest
Hello, @whirlxd! Thanks for contributing to the project!
It looks like your PR didn't pass the auto checks. If it's not difficult, please fix what is described in sections 8, 9, 10, 11 in CONTRIBUTING.md. Or I can do it myself.
I will also write some comments from the review below.
It seems it's not prettier formatted, i dont use prettier could you do that
Hello @whirlxd we already have prettier installed as package in our repo
You can run
npm run fmt
commit changes and push them to this PR
hey sorry i got busy , it should be formatted now
Hello, @whirlxd!
If it's not difficult, please fix what is described in sections 8, 9 in CONTRIBUTING.md. Or I can do it myself.
I also wrote some comments on the code review. Please fix them.
Hello, @whirlxd! I hope you're doing well. I noticed that the German translation still has some points we discussed. If you don't have time, you can just answer my questions related to translation directly in the PR and I'll make the necessary corrections myself. If something is unclear, we'll be happy to explain!
Thank you!
Yo actually i got really busy and forgot about this. I would appreciate if you could make those changes and merge though And once again really sorry
Yo actually i got really busy and forgot about this. I would appreciate if you could make those changes and merge though And once again really sorry
@whirlxd, thanks for answering! I'll make the changes myself, but could you please answer the translation questions directly in the PR?
day
in German? For example, in English, we have: (0) Days, (1) Day, (2-inf) Days. What are the German forms for these counts?1 - tag , rahe, tag 2-aktueller zyklustag 3- erster tag ( 1st day), zweiter tag ( 2) and so on just add an er at the end
@whirlxd, thanks for helping with the translation into German! And congratulations on your first contribution to the Peri
project!
glad to have helped
added trans for german