Skip to content
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

Tłumaczenie #2

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Tłumaczenie #2

wants to merge 7 commits into from

Conversation

dawidbil
Copy link
Contributor

No description provided.

Copy link

@Penthurion Penthurion left a comment

Choose a reason for hiding this comment

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

Chyba przypadkowo Ci się zrobił komentarz zamiast tagu 😄 . Jeśli jednak zabieg był zamierzony to proszę zignoruj moją sugestię. Poza tym nie widziałem żadnych problemów.

Comment on lines +115 to +118
<!-- section data-markdown="06_stl_algo_iter.en.md"
data-separator-vertical="^___"
data-separator-notes="^Note:">
</section-->

Choose a reason for hiding this comment

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

Suggested change
<!-- section data-markdown="06_stl_algo_iter.en.md"
data-separator-vertical="^___"
data-separator-notes="^Note:">
</section-->
<section data-markdown="06_stl_algo_iter.en.md"
data-separator-vertical="^___"
data-separator-notes="^Note:">
</section>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No właśnie ten komentarz był wcześniej i nie wiedziałem co zrobić, więc zostawiłem tak jak jest 😃.
Wersję angielską tylko automatycznie wygenerowałem i zostawiłem.

@dawidbil dawidbil requested a review from ziobron as a code owner May 5, 2021 10:24
@ziobron
Copy link
Contributor

ziobron commented May 5, 2021

Ja na razie tego nie mogę przejrzeć, bo mam niestety inne priorytety i standardowo (a może nawet bardziej niż zwykle) brak czasu.
Ale zmieniła się cała struktura repozytorium.
Chciałbyś @MrMijagi dostosować to do nowej struktury? Pliki pozmieniały nazwy, lokalizacje, niektóre rzeczy zostały podzielone. W sumie to bardziej podpada pod robotę od nowa :/

@dawidbil
Copy link
Contributor Author

dawidbil commented May 5, 2021

@ziobron
Ok, przerobię to, nie powinno być tak dużo roboty 🙂. Co do nazw plików, mam zastosować tą samą konwencję w stylu polskie wersje z dopiskiem ".pl.md" oraz angielskie ".en.md"?

@ziobron
Copy link
Contributor

ziobron commented May 5, 2021

Nie :) Zrób nowego brancha i zostaw nazwy te same co obecnie, bez pl i en. Po prostu na master będzie polska wersja, a na branchu english angielska :)

apietrzyk92 added a commit to apietrzyk92/cpp-fundamentals that referenced this pull request Jan 8, 2023
apietrzyk92 added a commit to apietrzyk92/cpp-fundamentals that referenced this pull request Jan 10, 2023
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.

None yet

3 participants