* Add RTL/LTR Markdown linter for mixed-direction text consistency and PR annotation
Introduce a Python-based linter (scripts/rtl_ltr_linter.py) to automatically detect and annotate issues related to mixed Right-To-Left (RTL) and Left-To-Right (LTR) text in Markdown files. The linter analyzes list items, book entries, and metadata for potential bidirectional text rendering problems, such as missing Unicode directionality markers (RLM/LRM) and improper handling of LTR keywords or symbols in RTL contexts.
Key features:
- Scans all Markdown files in the repository, with full logs saved as workflow artifacts.
- Annotates only changed or added lines in pull requests, providing targeted feedback in the GitHub Actions Job Summary.
- Detects common RTL/LTR issues, including:
- Missing directionality markers after LTR keywords (e.g., "HTML") or symbols (e.g., "C#") in RTL text.
- BIDI (bidirectional) mismatches that may affect text display.
- Incorrect ordering of author names and metadata in RTL contexts.
- Configurable via rtl_linter_config.yml for keywords, symbols, and severity levels.
- Includes a GitHub Actions workflow (rtl-ltr-linter.yml) for automated checks on PRs.
* Add test cases for RTL/LTR linter in English and Arabic book lists
Add sample entries to free-programming-books-en.md and free-programming-books-ar.md to test the RTL/LTR Markdown linter.
These test cases include various combinations of RTL and LTR text, keywords, symbols, and metadata to verify that the linter correctly detects directionality issues and outputs the expected logs and annotations.
* Restore original book lists after RTL/LTR linter test cases
Revert test entries in free-programming-books-en.md and free-programming-books-ar.md, restoring the original book lists. This commit removes temporary test data used for validating the RTL/LTR Markdown linter, preparing the repository for merging the PR with a clean state.
No functional changes to the linter or configuration files; only test content has been removed.
* Update RTL/LTR linter workflow and script: run only on RTL file changes or "RTL" label, fail only on errors
The GitHub Actions workflow for the RTL/LTR Markdown linter now runs only if:
- The PR modifies .md files related to RTL languages (ar, he, fa, ur), or
- The PR has the "RTL" label.
The linter script has been updated to fail the check only if errors are found on changed lines, not for warnings.
* Only upload linter artifact if linter step runs
Prevent warning about missing artifact by uploading the linter output log only if the linter step was executed (success or failure). This avoids unnecessary warnings when the linter is skipped because no RTL files were changed and no RTL label is present.
* Test workflow: modify non-RTL markdown file
Modified free-programming-books-en.md to verify that the RTL/LTR linter workflow does not run when only non-RTL markdown files are changed and the "RTL" label is not present.
* Test workflow: modify RTL markdown file to trigger linter
Modified free-programming-books-ar.md to verify that the RTL/LTR linter workflow runs as expected when an RTL markdown file is changed.
* Fix RTL/LTR BIDI issues in some markdown files
Applied directional markers (‎, ‏) and other formatting fixes to resolve BIDI (bidirectional text) errors and warnings reported by the linter in several .md files.
* Fix workflow: upload linter log only if linter step has not been skipped
Updated the workflow to upload the linter output artifact only when the linter step was actually executed (not skipped)
* Add debug step to check linter outcome in workflow
Added a debug step after the linter execution in the workflow to print the outcome and conclusion of the run_linter step.
* Set continue-on-error for linter step to allow artifact upload and debug
* Remove workflow debug step and update markdown file
Removed the debug step from the RTL/LTR linter workflow and applied further changes to a markdown file.
* Fix RTL/LTR BIDI issues in some markdown files
Applied directional markers (‎, ‏) and other formatting fixes to resolve BIDI (bidirectional text) errors and warnings reported by the linter in several .md files. This commit is a second batch of corrections to improve RTL/LTR rendering and pass the linter checks.
* Fix RTL/LTR BIDI issues in some markdown files
Applied directional markers (‎, ‏) and other formatting fixes to resolve BIDI (bidirectional text) errors and warnings reported by the linter in several .md files. This commit is a third batch of corrections to improve RTL/LTR rendering and pass the linter checks.
* Do not produce log file if no issues found
Updated the linter script to avoid creating the log file when no issues, warnings, or notices are found. If no issues are detected, the script now prints a "::notice ::No issues found"
* Always print annotation with number of errors and warnings found
Updated the linter script to always print an annotation indicating how many errors and warnings were found, even if there are none.
* Fix: always print summary annotation with number of issues found
* Add a missing newline character at end of file free-courses-he.md
* Update linter configuration and revert markdown files to pre-fix state
Updated the organization of keywords and symbols in the linter configuration file. Reverted all markdown files to their original state prior to the fixes.
* Update free-programming-books-he.md with fixes
* Update free-programming-books-he.md with further fixes
* Update free-programming-books-he.md with fixes
* Update free-programming-books-fa_IR.md with fixes
* Update free-programming-books-he.md with further fixes
* Update free-programming-books-ar.md with fixes
* Update free-programming-books-ar.md with further fixes
* Update free-podcasts-screencasts-ar.md with fixes
* Update free-podcasts-screencasts-fa_IR.md with fixes
* Update free-courses-he.md with fixes
* Update free-courses-he.md with further fixes
* Update free-courses-fa_IR.md with fixes
* Update free-courses-fa_IR.md with further fixes
* Update free-courses-ar.md with fixes
* Update free-courses-ar.md with further fixes
* Update free-courses-ar.md with further fixes
* Update free-courses-ur.md with fixes
* Update some markdown files with further improvements
* Fix alignment of nested lists in free-programming-books-fa_IR.md
* Update CONTRIBUTING.md and CONTRIBUTING-it.md with RTL/LTR linter error fixing guidelines
Added a section to CONTRIBUTING.md and CONTRIBUTING-it.md explaining how to fix RTL/LTR Markdown linter errors, including when to use ‏ and ‎ with practical examples for contributors working on files with mixed RTL and LTR text
* Update free-programming-books-fa_IR.md
The Tao of Programming
* Update free-programming-books-fa_IR.md
Reorder alphabetically.
* fix PDF indication
add pdf type of file
* fix space in pdf indication
fix space in pdf indication
* Update free-programming-books-fa_IR.md
fix typo
* Update free-programming-books-fa_IR.md
The Tao of Programming
* Update free-programming-books-fa_IR.md
Reorder alphabetically.
* fix PDF indication
add pdf type of file
* fix space in pdf indication
fix space in pdf indication
* Update free-programming-books-fa_IR.md
Add semi-space to the Persian book list
* Update CODE_OF_CONDUCT-fa_IR.md
Add semi-space and translation fix to the Persian contribution guide
* Update free-programming-books-fa_IR.md
fix number of blank lines before the heading
* improve the transaltion
* resources format is title-author, not author-title
* fix: link inside link is not valid markdown syntax
it seems that this resource was two in the past. recovering offending resource from #5273
* fix: resources format stars with a link. deduplicate both versions
* complete creators of altered resources
* fix: homogenize authors format and place
- use colon as author separator (replaces `&`, `and`, `y`, `e`, `et`...)
- ensures is placed after resource title, not part of itself `title - author`
- `et al.` as special author/token to group many
* remove double comma
* fix: recover traducer role notations as `trad.:`
* detect and apply some `trad.:`s role anotation
* chore: `trad.:` to `trl.:`
Apply suggestion made by Eric at https://github.com/EbookFoundation/free-programming-books/pull/7034#discussion_r954014539
Co-authored-by: Eric Hellman <eric@hellman.net>
* Sort author - format
* fix: remove empty authors (dash is alone)
Used regex: `-\s+(\(|$)`
* format: change to `edt.:` role
Definition list at https://www.loc.gov/marc/relators/relaterm.html
Co-authored-by: Eric Hellman <eric@hellman.net>
* chore: homogenize C# table of content anchor (`#c-sharp` to `#csharp`)
* format: homogenize c# category title
right markdown escape instead of use HTML entities
* lint: fix `MD012/no-multiple-blanks` rule
Multiple consecutive blank lines [Expected: 2; Actual: 3]
* lint: fix `MD039/no-space-in-links`
Spaces inside link text
* lint: fix `MD009/no-trailing-spaces` rule
Trailing spaces [Expected: 0 or 4; Actual: 1]
Trailing spaces [Expected: 0 or 4; Actual: 2]
* lint: fix `MD006/ul-start-left` rule
* format: remove extra spaces between note tokens
* spell: fix lint `MD044/proper-names` rule
- HTML
- JavaScript
- YouTube
* spell: normalize to the most common `Index` heading
* lint: fix `MD007/ul-indent` rule
Unordered list indentation [Expected: 4; Actual: 2]
* format: escape pipes `|` from resources text
addresses #5176
* add Hesham Asem courses about machine and deep learning and nlp
* Expand the name of nlp section
* Change the order
* Revert "spell: normalize to the most common `Index` heading"
This reverts commit 4d6a74e7d794f1f264cb2f88d9531e43038d3be2.
Co-authored-by: AhmedElTabarani <eltabaraniahmed@gmail.com>
* add react js into js section
* fix order with swaping
* Update books/free-programming-books-fa_IR.md
Co-authored-by: David Ordás <3125580+davorpa@users.noreply.github.com>
* fix: alphabetize titles first, after creators
Co-authored-by: Eric Hellman <eric@hellman.net>
Co-authored-by: David Ordás <3125580+davorpa@users.noreply.github.com>
* New books on linux added
* New Git course added
* signup required (in Farse) added to the course
* fix linter issue
Co-authored-by: Emad Ehsanrad <emad.ehsanrad@gmail.com>
