Skip to content

Input and Output#20

Open
TrifanBogdan24 wants to merge 13 commits into
UPB-CS-OpenSourceUpstream:masterfrom
TrifanBogdan24:new-chapter
Open

Input and Output#20
TrifanBogdan24 wants to merge 13 commits into
UPB-CS-OpenSourceUpstream:masterfrom
TrifanBogdan24:new-chapter

Conversation

@TrifanBogdan24

@TrifanBogdan24 TrifanBogdan24 commented Jan 8, 2024

Copy link
Copy Markdown

Discussed topics

  • printing text
  • writing text
  • reading text from console
  • command line arguments
  • LINUX environment variables

Solved issue:

The new explanations are described both in English and Romanian.

@TrifanBogdan24 TrifanBogdan24 marked this pull request as ready for review January 8, 2024 17:01
@TrifanBogdan24

Copy link
Copy Markdown
Author

All the other chapters are not modified

@TrifanBogdan24 TrifanBogdan24 mentioned this pull request Jan 8, 2024
Comment thread lessons/en/chapter_10.yaml Outdated
Comment thread lessons/en/chapter_10.yaml
@TrifanBogdan24

Copy link
Copy Markdown
Author

Thank you D9nni for the feedback, I hope it is all right now.

@alexandruradovici

Copy link
Copy Markdown

State in the description that you added a chapter to the tutorials.

@alexandruradovici alexandruradovici left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Please modify the translations for chapter 10 also.

Comment thread lessons/en/chapter_10.yaml Outdated
Comment thread lessons/en/chapter_10.yaml Outdated
Comment thread lessons/en/chapter_10.yaml Outdated
Comment thread lessons/en/chapter_10.yaml Outdated
Comment thread lessons/en/chapter_10.yaml Outdated
Comment thread lessons/en/chapter_10.yaml
Comment thread lessons/en/chapter_10.yaml
Comment thread lessons/en/chapter_10.yaml
Comment thread lessons/en/chapter_10.yaml
Comment thread lessons/en/chapter_10.yaml

@D9nni D9nni left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review for romanian translation.

Comment thread lessons/ro/chapter_10.yaml Outdated
Comment thread lessons/ro/chapter_10.yaml Outdated
Comment thread lessons/ro/chapter_10.yaml Outdated
Comment thread lessons/ro/chapter_10.yaml Outdated
Comment thread lessons/ro/chapter_10.yaml Outdated
Comment thread lessons/ro/chapter_10.yaml Outdated
Comment thread lessons/ro/chapter_10.yaml Outdated
Comment thread lessons/ro/chapter_10.yaml Outdated
Comment thread lessons/ro/chapter_10.yaml Outdated
Comment thread lessons/ro/chapter_10.yaml Outdated
Comment thread lessons/ro/chapter_10.yaml Outdated
Comment thread lessons/ro/chapter_10.yaml Outdated
Comment thread lessons/en/chapter_10.yaml Outdated
Comment thread lessons/en/chapter_10.yaml Outdated
@alexandruradovici

Copy link
Copy Markdown

@TrifanBogdan24 please mark the changes that you made as resolved.

@alexandruradovici alexandruradovici left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Please rename all the chapter_10.yaml files to chapter_11.yaml as all the translations have to shift.

@alexandruradovici alexandruradovici left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Please rebase this so you can send it to upstream.

@TrifanBogdan24

Copy link
Copy Markdown
Author

I hope that my branch new-chapter is rebased

@TrifanBogdan24

Copy link
Copy Markdown
Author

Here is the link to the upstream: richardanaya#516

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.

5 participants