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

Linux cleanup #3992

Merged
merged 6 commits into from
Jul 12, 2022
Merged

Linux cleanup #3992

merged 6 commits into from
Jul 12, 2022

Conversation

jpleboeuf
Copy link
Contributor

@jpleboeuf jpleboeuf commented Jul 12, 2022

Hello,

here are some fixes for problems I encountered when training with the Linux quiz:

In brief:

  • move similar questions Q17&Q45 next to each other, with notes to point these questions are similar; also provide the correct answer to Q45, and a side note about a potential compatibility problem;
  • provide the correct answer to Q52, add the reference and a quote;
  • fix a typo in Q84;
  • fix text in Q89 and Q90 (and also the link formatting in Q89);
  • and finally, automatically renumber questions after last changes.

Pings: @D4RIO, @Amsal1.

Hope it helps,
and thanks again for this project!

Q45 is similar to Q17.
Moved Q45 next to Q17, adding notes
 so that it's easier to grasp the questions are similar.

Also, fixed Q45,
 by providing the correct answer,
 and adding a side note about potential compatibility problems.
Re-answered Q52 correctly:
 with "no-hang-up",
 the script will run in the background,
 even after the end of the session.

Added a link to the reference, with a quote.
In Q84, the word "extended" had a typo.
There were strange copy/paste bombs in Q89, fixed.

Also, fixed the Markdown for the reference provided.
There were strange copy/paste bombs in Q90, fixed.
Automatically renumbered questions because of last moves.
Copy link
Owner

@Ebazhanov Ebazhanov left a comment

Choose a reason for hiding this comment

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

Thank you @jpleboeuf 🚀

@Ebazhanov Ebazhanov merged commit e59e66a into Ebazhanov:main Jul 12, 2022
@D4RIO
Copy link
Contributor

D4RIO commented Jul 13, 2022

Just came to say everything seems fine, thank u for the atomic, meaningful commits.

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