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

Add new collection of Arabic Courses #11103

Merged

Conversation

AhmedElTabarani
Copy link
Contributor

What does this PR do?

Add resource(s) | Remove resource(s) | Add info | Improve repo

Hi, It was a while since the last PR :)
here are new Arabic courses

For resources

Description

  • I create a new section called Cloud Computing and move AWS inside it
  • I create a new section called Computer Science and move some courses from Algorithms & Data Structures to it
    Courses related with (CS In General, Fundamentals/introduction To Computers, How Computer Works, What we should know before leaning programming, ..etc.)
  • I add new section called Nest.js
  • Add a lot of courses in different topic

Why is this valuable (or not)?

N/A

How do we know it's really free?

YouTube

For book lists, is it a book? For course lists, is it a course? etc.

course lists

Checklist:

  • Read our contributing guidelines.
  • Search for duplicates.
  • Include author(s) and platform where appropriate.
  • Put lists in alphabetical order, correct spacing.
  • Add needed indications (PDF, access notes, under construction).
  • Used an informative name for this pull request.

Follow-up

  • Check the status of GitHub Actions and resolve any reported warnings!

Copy link

Linter failed, fix the error(s):

free-programming-books/courses/free-courses-ar.md
   29:5-36:23  warning  Alphabetical ordering: swap l.32 and l.31                              alphabetize-lists  remark-lint
  58:1-75:162  warning  Alphabetical ordering: swap l.63 and l.62                              alphabetize-lists  remark-lint
101:1-113:122  warning  Alphabetical ordering: swap l.102 and l.101                            alphabetize-lists  remark-lint
144:1-145:113  warning  Alphabetical ordering: swap l.145 and l.144                            alphabetize-lists  remark-lint
161:1-171:180  warning  Alphabetical ordering: swap l.162 and l.161                            alphabetize-lists  remark-lint
161:1-171:180  warning  Incorrect number of blank lines between last section and next heading  blank-lines-1-0-2  remark-lint
215:1-217:135  warning  Incorrect number of blank lines between last section and next heading  blank-lines-1-0-2  remark-lint
388:1-401:154  warning  Alphabetical ordering: swap l.390 and l.389                            alphabetize-lists  remark-lint
515:1-524:119  warning  Alphabetical ordering: swap l.518 and l.517                            alphabetize-lists  remark-lint
566:1-581:179  warning  Alphabetical ordering: swap l.577 and l.576                            alphabetize-lists  remark-lint

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

Linter failed, fix the error(s):

free-programming-books/courses/free-courses-ar.md
390:1-403:154  warning  Alphabetical ordering: swap l.393 and l.392  alphabetize-lists  remark-lint
517:1-526:119  warning  Alphabetical ordering: swap l.520 and l.519  alphabetize-lists  remark-lint
568:1-583:179  warning  Alphabetical ordering: swap l.579 and l.578  alphabetize-lists  remark-lint

Copy link

Linter failed, fix the error(s):

free-programming-books/courses/free-courses-ar.md
568:1-583:179  warning  Alphabetical ordering: swap l.580 and l.579  alphabetize-lists  remark-lint

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

Very nice work! For future reference, PR's are easier to review when they're separated into chunks by what they do. For example, you might revise one section at a time, or separate the reorganization from the resource editing.

@eshellman eshellman merged commit 877d372 into EbookFoundation:main Mar 24, 2024
5 checks passed
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.

2 participants