Skip to content

Add Polish translation #63

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

Merged
merged 174 commits into from
Apr 4, 2020
Merged

Add Polish translation #63

merged 174 commits into from
Apr 4, 2020

Conversation

mbiesiad
Copy link
Contributor

@mbiesiad mbiesiad commented Mar 10, 2020

Hi there! First of all, really great project. Polish translation of docs is [WIP]: https://github.com/mbiesiad/ava-docs/tree/pl_PL

Stay tuned, best regards.

@mbiesiad
Copy link
Contributor Author

mbiesiad commented Apr 2, 2020

@kekuu Could you take a look & CR?

@kekuu
Copy link

kekuu commented Apr 4, 2020

sure, looks OK

Copy link

@kekuu kekuu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

- Nowym funkcjom muszą towarzyszyć testy i dokumentacja (w języku angielskim).
- Nie dodawaj zmian niezwiązanych z pull requestem.
- Sprawdź (Lint) i przetestuj przed wysłaniem pull requestem, uruchamiając `$ npm test`.
- Złóż pull request [branche](https://github.com/dchelimsky/rspec/wiki/Topic-Branches), nie dla mastera.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe better will be 'wyślij pull request'

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure, agree! Thanks!


Należy pamiętać, że ten projekt został opublikowany z [Code of conduct](code-of-conduct.md). Uczestnicząc w tym projekcie, zgadzasz się przestrzegać jego warunków.

Tłumaczenia : [English](https://github.com/avajs/ava/blob/master/contributing.md), [Español](https://github.com/avajs/ava-docs/blob/master/es_ES/contributing.md), [Italiano](https://github.com/avajs/ava-docs/blob/master/it_IT/contributing.md), [日本語](https://github.com/avajs/ava-docs/blob/master/ja_JP/contributing.md), [Português](https://github.com/avajs/ava-docs/blob/master/pt_BR/contributing.md), [Русский](https://github.com/avajs/ava-docs/blob/master/ru_RU/contributing.md), [简体中文](https://github.com/avajs/ava-docs/blob/master/zh_CN/contributing.md)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe better without white space -> "Tłumaczenia:"


Jako użytkownik AVA jesteś idealnym kandydatem, aby pomóc nam ulepszyć naszą dokumentację: poprawki literówek, poprawki błędów, lepsze wyjaśnienia, więcej przykładów itp. Otwarte problemy dotyczące rzeczy, które można poprawić. [Pomóż przetłumaczyć nasze dokumenty.](https://github.com/avajs/ava-docs) Cokolwiek, nawet ulepszenia wprowadzone w tym dokumencie.

Użyj [etykiety `docs`](https://github.com/avajs/ava/labels/docs) aby znaleźć sugestie dotyczące tego, co chcielibyśmy zobaczyć więcej w dokumentacji.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

generally I think both correct co/czego, but maybe better sounds 'czego' here


*Czy chcesz wnieść swój pierwszy wkład w projekt typu open source? Nie szukaj dalej! AVA może być jednym z najbardziej przyjaznych projektów i społeczności. Przeczytaj post na blogu ["Making your first contribution"](https://medium.com/@vadimdemedes/making-your-first-contribution-de6576ddb190) aby zacząć poprawnie i zrobić swój pierwszy udział w AVA !*

//Ten dokument tłumaczony został z języka francuskiego
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tłumaczony -> przetłumaczony

Copy link

@kekuu kekuu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@novemberborn
Copy link
Member

@mbiesiad thanks for all the activity here! @kekuu thanks for reviewing!

I take it this is good to go?

@mbiesiad
Copy link
Contributor Author

mbiesiad commented Apr 4, 2020

@novemberborn please wait, a second. I will check again & I will let you know soon

@mbiesiad
Copy link
Contributor Author

mbiesiad commented Apr 4, 2020

PS. By the way, I was even thinking about create an issue, but I will let you know here - the link for
'JSPM and SystemJS' is unfortunately invalid - official docs (recipes): https://github.com/avajs/ava here: https://github.com/avajs/ava/blob/master/docs/recipes/jspm-systemjs.md

@novemberborn
Copy link
Member

I was even thinking about create an issue, but I will let you know here

Issue or PR welcome 😉

mbiesiad added 4 commits April 4, 2020 15:06
fixed internal links for language version
fixed internal link
fixed internal link
updated contributing link
@mbiesiad
Copy link
Contributor Author

mbiesiad commented Apr 4, 2020

@mbiesiad thanks for all the activity here! @kekuu thanks for reviewing!

I take it this is good to go?

@novemberborn looks OK, yup.

@mbiesiad
Copy link
Contributor Author

mbiesiad commented Apr 4, 2020

I was even thinking about create an issue, but I will let you know here

Issue or PR welcome 😉

done :-)

@novemberborn novemberborn changed the title [WIP] Polish version Add Polish translation Apr 4, 2020
@novemberborn novemberborn merged commit 18cbe3b into avajs:master Apr 4, 2020
@novemberborn
Copy link
Member

Awesome, thanks!

@mbiesiad
Copy link
Contributor Author

mbiesiad commented Apr 4, 2020

my pleasure! Best wishes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants