-
-
Notifications
You must be signed in to change notification settings - Fork 17
Translate file licensce.po #138
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
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.
@skpanagiotis πάρα πολύ καλή δουλειά και ένα από τα πιο δύσκολα αρχεία! Έχω αφήσει κάποια suggestions αν θέλεις ρίξε μια ματιά.
Δύο κύρια σχόλια έχω γενικά και θα ήθελα την γνώμη σου σχετικά με την μετάφραση ή μη του included (βλ. σχόλιο παρακάτω) και του test suite.
@skpanagiotis συγκεκριμένο PR επιλύει και το issue #15. Αν θέλεις κάνε ένα comment και εκεί για να στο κάνω assign. Το μόνο που μένει στο συγκεκριμένο PR είναι να αφαιρεθεί τελείως το LICENSE
@thepetk σε Ευχαριστώ πολύ για τα σχόλια, θα τα διορθώσω και θα είμαι λίγο πιο προσεκτικός σε απροσεξίες και μικρολαθάκια. Όσο για τις λέξεις test suite και included, παρατήρησα ότι η μετάφραση στα ισπανικά δεν αφήνει τις λέξεις ίδιες αλλά τις μεταφράζει. Για αυτό έκανα την μετάφραση.
@thepetk σε Ευχαριστώ πολύ για τα σχόλια, θα τα διορθώσω και θα είμαι λίγο πιο προσεκτικός σε απροσεξίες και μικρολαθάκια. Όσο για τις λέξεις test suite και included, παρατήρησα ότι η μετάφραση στα ισπανικά δεν αφήνει τις λέξεις ίδιες αλλά τις μεταφράζει. Για αυτό έκανα την μετάφραση.
Καλά εννοείται πως όλοι κάνουμε μικρολαθάκια! Νομίζω είναι καλό point για την ισπανική μετάφραση άρα ας το κρατήσουμε έτσι. Νομίζω θα πρέπει να τρέξεις και το powrap για το license.po (το test workflow αποτυγχάνει για αυτό).
Έκανα τις αλλαγές που μου υπέδειξες όπως επίσης διέγραψα το LICENSE σύμφωνα με το issue #15.
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.
lgtm
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.
Άφησα ένα πολύ μικρό suggestion. Επίσης μπορείς να βάλεις τα στοιχεία σου εδώ (αν θελεις):
Άφησα ένα πολύ μικρό suggestion. Επίσης μπορείς να βάλεις τα στοιχεία σου εδώ (αν θελεις):
Όπως και δω
https://github.com/pygreece/python-docs-gr/blob/main/TRANSLATORS
@thepetk έκανα αυτά που μου πρότεινες
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.
lgtm
Ποιο issue κλείνει το συγκεκριμένο PR;
Closes #16
Περιγραφή του PR
Μετάφραση του αρχείου license.po.