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

Update free-programming-books-fa_IR.md #11101

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

man4toman
Copy link

@man4toman man4toman commented Mar 16, 2024

What does this PR do?

Add some resource(s)

Copy link

Linter failed, fix the error(s):

free-programming-books/books/free-programming-books-fa_IR.md
48:1-51:56  warning  Alphabetical ordering: swap l.51 and l.50           alphabetize-lists      remark-lint
51:3-51:56  warning  Remove trailing slash (https://fa.javascript.info)  no-url-trailing-slash  remark-lint
75:1-77:97  warning  Alphabetical ordering: swap l.77 and l.76           alphabetize-lists      remark-lint

@github-actions github-actions bot added the linter error Please, correct build errors found by linter! label Mar 16, 2024
@eshellman
Copy link
Collaborator

Be sure to check the free-programming-books-lint output for linter errors. Alphabetical order and spacing are the most common problems.

Copy link
Contributor

@PiousSutherland PiousSutherland left a comment

Choose a reason for hiding this comment

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

Changes to be made

[!NOTE]
I did not check the links.


Lines 50&51 need to be swapped, with trailing / removed; fix:

* [آموزش جاوااسکریپت مدرن](https://fa.javascript.info)
* [ریکت جی اس](https://github.com/reactjs/fa.reactjs.org).

Lines 75-77 need to be updated:

* [آموزش برنامه نویسی شئ گرایی در پایتون](https://github.com/persianpython/PythonObjectOriented)
* [ترجمه آزاد کتاب Asyncio in Python](https://github.com/ftg-iran/aip-persian)
* [پایتون به پارسی](https://python.coderz.ir) - سعید درویش (HTML)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
linter error Please, correct build errors found by linter!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants