Skip to content

Syria language#108432

Closed
Ghaith-Shammout wants to merge 9 commits into
firstcontributions:mainfrom
Ghaith-Shammout:syria-language
Closed

Syria language#108432
Ghaith-Shammout wants to merge 9 commits into
firstcontributions:mainfrom
Ghaith-Shammout:syria-language

Conversation

@Ghaith-Shammout
Copy link
Copy Markdown

@Ghaith-Shammout Ghaith-Shammout commented Dec 8, 2025

Before submitting this pull request, check the changes to see it's only the changes you made intentionally
If there are changes to other lines you didn't make deliberately, it's possible that your IDE made the changes with a utility like prettier.
Next time, make sure that you only add your changes by using git add -p and rather than git add Contributors.md

If you're doing something in the checklist below, put an x inside [ ] so that - [ ] becomes - [x]

  • I had fun going through this tutorial (ノ^o^)ノ and learned on the way ٩(^◡^)۶
  • There are some things I'd like to improve in this tutorial. I have written them below.
  • There were steps where I had errors while following this tutorial. I have written them below.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 8, 2025

Thank you for your pull request. This pull request contains changes in files which requires review. The following files were changed:

  • Contributors.md
  • README.md
  • docs/translations/README.apc.md

Copy link
Copy Markdown
Member

@rammba rammba left a comment

Choose a reason for hiding this comment

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

Hello @Ghaith-Shammout, thanks for the contribution and welcome to this repository 🚀

It looks like you didn't translate whole file and that sy language code doesn't exist. Can you please check my comments?

@@ -0,0 +1,87 @@
# 🇸🇾 أول مساهمة إلك بهالمشروع
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Which language is this exactly?
Sorry if I'm missing something, but it looks like there is no official Syrian language, or at least I didn't found it by quick googling. Wikipedia article says that there are few languages spoken in Syria.

Comment thread docs/translations/README.sy.md Outdated
@@ -0,0 +1,87 @@
# 🇸🇾 أول مساهمة إلك بهالمشروع

هالمشروع هدفه يسهّل الطريق للمبتدئين ليعملوا أول مساهمة إلهم بالمشاريع مفتوحة المصدر. إذا بدك تبلّش وتعمل أول مساهمة، اتبع الخطوات الجاية:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks like you didn't translate whole official readme file. Few sections are missing and non of the images are here.

Comment thread README.md Outdated
#### _Read this in [other languages](docs/translations/Translations.md)._


<kbd>[<img title="Syria" alt="Syria" src="https://upload.wikimedia.org/wikipedia/commons/5/54/Flag_of_Syria_%282025-%29.svg" width="22">](docs/translations/README.sy.md)</kbd>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

New languages often goes at the end of the list. I think we will sort the list at some moment.

@Ghaith-Shammout
Copy link
Copy Markdown
Author

Hi @rammba and thanks for your help on this PR.
I updated the branch based on your feedback.

When it comes to the language, it should be Arabic with Syria dialect. I missed to type Arabic because I was very exited to contribute to this repo.

Kindly, let me know if there anything that need further attention

Copy link
Copy Markdown
Member

@rammba rammba left a comment

Choose a reason for hiding this comment

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

Hello @Ghaith-Shammout, thanks for clarification.
If it's Arabic language with Syrian dialect, and not some other eg. Levantine Arabic, can you please rename file to README.ar-sy.md then?
Sorry if I'm picky about this, but we already have an issue (#105711) to properly name existing languages, so it's good to have correct names in the new languages.

@Ghaith-Shammout
Copy link
Copy Markdown
Author

Not at all, I like getting feedback 😊
What if I want this contribution to be in Levantine Arabic, what should I call the README file, what additional step do I need to take?

@rammba
Copy link
Copy Markdown
Member

rammba commented Dec 10, 2025

Hello @Ghaith-Shammout, thanks for the interest in the proper naming of the file.
I'm not familiar with variations of Arabic languages at all, but, in theory, it's possible to add both Syrian Arabic and Levantine Arabic, if they are not the same.
As mentioned in #105711, we want to name files with proper language code based on ISO 639 list. So, if you want to use Levantine, use apc code, or ar-sy for Syrian Arabic.

@Roshanjossey
Copy link
Copy Markdown
Member

@Ghaith-Shammout
Copy link
Copy Markdown
Author

Ghaith-Shammout commented Dec 17, 2025

Yes, I do understand the content in that README file @Roshanjossey.
I just want to contribute to this repo by adding the Levantine Arabic.

Copy link
Copy Markdown
Member

@rammba rammba left a comment

Choose a reason for hiding this comment

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

Hello @Ghaith-Shammout, thanks for the clarification about used language.
Can you maybe add yourself to the CONTRIBUTING.md. Doing that, everyone can find you and ask for help about future Levantine Arabic translations.

Comment thread README.md Outdated
<kbd>[<img title="Mongolian" alt="Mongolian" src="https://cdn.statically.io/flags/mn.svg" width="22">](docs/translations/README.mn.md)</kbd>
<kbd>[<img title="Tounsi" alt="Tounsi" src="https://cdn.statically.io/gh/hjnilsson/country-flags/master/svg/tn.svg" width="22">](docs/translations/README.tn.md)</kbd>
<kbd>[<img title="Lingala" alt="Lingala" src="https://cdn.statically.io/gh/hjnilsson/country-flags/master/svg/cd.svg" width="22">](docs/translations/README.ln.md)</kbd>
<kbd>[<img title="Syria" alt="Syria" src="https://upload.wikimedia.org/wikipedia/commons/5/54/Flag_of_Syria_%282025-%29.svg" width="22">](docs/translations/README.sy.md)</kbd>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Change this line to use Levantine Arabic and correct README file name.

@rammba
Copy link
Copy Markdown
Member

rammba commented Dec 17, 2025

Hello @Roshanjossey, you can merge new language, Levantine Arabic 🚀

@Ghaith-Shammout
Copy link
Copy Markdown
Author

Thank you for yours support 🤍

@Roshanjossey
Copy link
Copy Markdown
Member

I asked my Syrian friends too. They can also understand https://github.com/firstcontributions/first-contributions/blob/main/docs/translations/README.ar.md. In this case, I think it's best not to add this translation.

@rammba
Copy link
Copy Markdown
Member

rammba commented Dec 22, 2025

Hello @Roshanjossey, even though they understand, maybe this language has varieties with the original Arabic.
I'm not familiar with Arabic, but Balkan languages (Serbian, Croatian, Bosnian, Montenegrin) are also a bit similar, and we have all of them listed. There are differences of course, but I think people can grasp the gist of the content in these languages.

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.

3 participants