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 27 commits into
base: main
Choose a base branch
from
Open

Tłumaczenie #2

wants to merge 27 commits into from

Conversation

Penthurion
Copy link

No description provided.

Copy link
Contributor

@dawidbil dawidbil left a comment

Choose a reason for hiding this comment

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

Oprócz drobnych zmian pliki 01_make.pl.md oraz 02_cmake.pl.md są po angielsku, poza tym wszystko wygląda w porządku 🙂

module1/01_make.en.md Show resolved Hide resolved
module1/02_cmake.en.md Show resolved Hide resolved

Napisz prosty CMakeLists.txt dla programu z katalogu `greeter`, zbuduj go za pomocą `cmake` i uruchom.
Write simple CMakeLists.txt for program from `greeter` directory, build it with `cmake` and run.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Write simple CMakeLists.txt for program from `greeter` directory, build it with `cmake` and run.
Write simple CMakeLists.txt for program from `greeter` directory, build it with `cmake` and run it.

module1/02_cmake.en.md Show resolved Hide resolved
<!-- .element: class="fragment fade-in" -->

Możemy za to bez większych problemów wykorzystać zmienną `${PROJECT_NAME}`. Zawiera ona nazwę projektu zdefiniowaną przez komendę `project()`
We can use the `$ {PROJECT_NAME}` variable without any problems. It contains the project name defined by the `project ()` command
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
We can use the `$ {PROJECT_NAME}` variable without any problems. It contains the project name defined by the `project ()` command
However, we can use the `$ {PROJECT_NAME}` variable without any problems. It contains the project name defined by the `project()` command

data-separator-vertical="^___"
data-separator-notes="^Note:">
</section>
<section data-markdown="presentation_cmake.md"
<section data-markdown="02_presentation_cmake.pl.md"
Copy link
Contributor

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="02_presentation_cmake.pl.md"
<section data-markdown="02_cmake.pl.md"

module1/index.pl.html Outdated Show resolved Hide resolved
module1/index.en.html Outdated Show resolved Hide resolved
module1/index.en.html Outdated Show resolved Hide resolved
module1/index.en.html Outdated Show resolved Hide resolved
Copy link
Contributor

@dawidbil dawidbil left a comment

Choose a reason for hiding this comment

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

Wygląda na to, że tylko w index.en.html są złe nazwy, reszta jest ok 🙂

module1/index.en.html Outdated Show resolved Hide resolved
module1/index.en.html Outdated Show resolved Hide resolved
module1/index.en.html Outdated Show resolved Hide resolved
PaSawi pushed a commit to PaSawi/cmake that referenced this pull request Mar 24, 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.

2 participants