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-ar.md #3675

Merged
merged 6 commits into from
Aug 13, 2020
Merged

Conversation

iAli
Copy link
Contributor

@iAli iAli commented Aug 12, 2020

What does this PR do?

Add Resource(s) | Remove Resource(s) | Improve Repo

For resources

Description

  • Fix the index
  • Add three new books
  • Update two books’ links

Why is this valuable (or not)

Useful for everyone looking for Arabic resources to learn programming

How do we know it's really free?

All of them are free and links from the original source

For book lists, is it a book?

Yes

Checklist:

  • Not a duplicate
  • Included author(s) if appropriate
  • Lists are in alphabetical order
  • Needed indications added (PDF, access notes, under construction)

Update ubuntu book’s link with the original source link
Update UX book’s link with the original source link
@eshellman
Copy link
Collaborator

Nice job, and welcome to free-programming-books! Remember, it's easier for us to review PRs if changes are broken up into smaller PRs and explained.

@eshellman eshellman merged commit 2ca5eb9 into EbookFoundation:master Aug 13, 2020
@eshellman
Copy link
Collaborator

(for example when a new source of multiple books is added, like academy.hsoub.com, it's useful to have an explanation of what kind of site it is - we like education sites, we don't like file-sharing sites.

@iAli
Copy link
Contributor Author

iAli commented Aug 13, 2020

Got it, I will make sure to do that next times. Thanks..

iAli added a commit to iAli/free-programming-books that referenced this pull request Aug 13, 2020
ignisc4t added a commit to ignisc4t/free-programming-books that referenced this pull request Aug 16, 2020
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