[ADD] Adding a blog post describing the review of "Introduction to the command line"#4
Merged
ultimatecoder merged 4 commits intoMar 3, 2017
Conversation
codingquark
reviewed
Mar 2, 2017
codingquark
left a comment
There was a problem hiding this comment.
This review only tries to fix language issues, and is not extensive. My comments could also be ambiguous.
| ## tl;dr | ||
|
|
||
|
|
||
| Every chapter will introduce a bunch of comandline utilities and will point to |
| its respective documentation for further learning. You should expect chapters | ||
| describing from the [grep](https://www.gnu.org/software/grep/manual/grep.html) | ||
| command to [GNU Octave](https://www.gnu.org/software/octave/) which is a | ||
| scientific programming language. The chapters are independent from each other. |
| describing from the [grep](https://www.gnu.org/software/grep/manual/grep.html) | ||
| command to [GNU Octave](https://www.gnu.org/software/octave/) which is a | ||
| scientific programming language. The chapters are independent from each other. | ||
| The book is must read if you new to the |
| command to [GNU Octave](https://www.gnu.org/software/octave/) which is a | ||
| scientific programming language. The chapters are independent from each other. | ||
| The book is must read if you new to the | ||
| [GNU/Linux](https://en.wikipedia.org/wiki/Linux) command line. If you new are at |
| The book is community driven and published under [FLOSS | ||
| Manual](http://flossmanuals.net). It is a collaborative effort of the | ||
| [FSF](http://www.fsf.org/) community. The fun part is you can contribute to this | ||
| book by adding new chapters or improve an existing one. I fixed one typo to this |
|
|
||
| I didn't encounter with any saviour mistakes except one typo. The section of | ||
| **Userful customizations** on page number 80 of my printed version, contains | ||
| below example: |
| ``` | ||
|
|
||
| Here, **$arg** is a misprint. A shell variable is never assigned with **$**. It | ||
| should be `args=$1`. I myself has corrected the typographical error in the |
|
|
||
| Here, **$arg** is a misprint. A shell variable is never assigned with **$**. It | ||
| should be `args=$1`. I myself has corrected the typographical error in the | ||
| book. This change will be published maybe in the next release of this book. |
| book. This change will be published maybe in the next release of this book. | ||
|
|
||
| If you are encountering any mistakes while reading, I request you to fix the | ||
| change to [here](http://write.flossmanuals.net/command-line/introduction/). The |
| If you are encountering any mistakes while reading, I request you to fix the | ||
| change to [here](http://write.flossmanuals.net/command-line/introduction/). The | ||
| interface for editing the book is beginner friendly. It took less than 5 minutes | ||
| to drive the change. |
There was a problem hiding this comment.
May be what you want to say is "make the change" instead of "drive the change"?
ultimatecoder
deleted the
master-bookreview-introduction-to-gnulinux-commandline
branch
December 5, 2017 12:10
ultimatecoder
pushed a commit
that referenced
this pull request
Jul 25, 2026
- scripts/build & scripts/test: add set -euo pipefail (and a shebang to scripts/test) so a failed build aborts CI instead of link-checking a stale _site (review #1) - scripts/test: move the --ignore-urls list into a documented bash array (review #5); note the baseurl<->swap-urls coupling in _config.yml (#6) - ci.yml: run bundle exec bundler-audit check --update as a real per-PR security scan (#2); add least-privilege permissions: contents: read (#3) - Pin all workflow actions to commit SHAs with version comments (#4) - Re-add minification via jekyll-minifier 0.2.2 (Jekyll 4-compatible, production-only) (#7) - Drop analytics: remove the leftover GA placeholder; UA died in 2023 (#8) - Fix 45 image references across 11 posts + about.md that used {{site.url}}/assets or hardcoded /assets and omitted baseurl, so they 404'd on the /Blog project-page URL. Verified via a production (minified) build + htmlproofer run with all internal links resolving.
ultimatecoder
added a commit
that referenced
this pull request
Jul 25, 2026
* Modernize build/deploy: Jekyll 4.4, GitHub Actions, GitHub Pages - Upgrade Jekyll 3.8.5 -> 4.4.1, pin Ruby 3.2.0, refresh gem dependencies and add x86_64-linux to the lockfile so CI can install on Linux runners - Add bundler-audit as a dev dependency; no known CVEs found - Fix scripts/test: htmlproofer was silently checking zero pages because the shell's missing LANG made Ruby default to US-ASCII; also drop the https-enforcement noise on old post links and ignore confirmed-dead external URLs - Retire the dead Travis CI / rsync-to-jaysinh.com pipeline (.travis.yml, scripts/deploy, CNAME, keypair.pem.enc) since that domain is no longer owned; the site now targets https://ultimatecoder.github.io/Blog/ with the matching /Blog baseurl - Add .github/workflows/ci.yml (build + link-check on PRs) and pages.yml (build + deploy to GitHub Pages on push to master) * Ignore two more external links that block GitHub Actions' IPs pythonexpress.in and hackerearth.com both return 200 from a normal network but 403 specifically from GitHub-hosted runner IPs (bot/WAF blocking, not a dead link) - confirmed via the ci.yml failure on PR #99. * Address PR review: fail-fast, security scan, minifier, image baseurl fix - scripts/build & scripts/test: add set -euo pipefail (and a shebang to scripts/test) so a failed build aborts CI instead of link-checking a stale _site (review #1) - scripts/test: move the --ignore-urls list into a documented bash array (review #5); note the baseurl<->swap-urls coupling in _config.yml (#6) - ci.yml: run bundle exec bundler-audit check --update as a real per-PR security scan (#2); add least-privilege permissions: contents: read (#3) - Pin all workflow actions to commit SHAs with version comments (#4) - Re-add minification via jekyll-minifier 0.2.2 (Jekyll 4-compatible, production-only) (#7) - Drop analytics: remove the leftover GA placeholder; UA died in 2023 (#8) - Fix 45 image references across 11 posts + about.md that used {{site.url}}/assets or hardcoded /assets and omitted baseurl, so they 404'd on the /Blog project-page URL. Verified via a production (minified) build + htmlproofer run with all internal links resolving. --------- Co-authored-by: Jaysinh Shukla <jaysinhshukla@Jaysinhs-MacBook-Pro-2.local>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description: Writing blog post reviewing book Introduction to the Command Line.
How to review this post?
Write your review directly to this pull request.
Review grammatical or spelling mistakes.
Suggest better sentence construction if you think any.
Write your review regarding the image linked with the blog post.