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

1 Mesajlaşma, 1 Harita uygulaması ve 1 Eklenti eklendi. #196

Merged
merged 21 commits into from
Dec 13, 2024

Conversation

denisz0
Copy link
Contributor

@denisz0 denisz0 commented Dec 13, 2024

No description provided.

docs/tarayici-eklentileri.md Outdated Show resolved Hide resolved
@GokturkTalha GokturkTalha self-requested a review December 13, 2024 19:44
@GokturkTalha
Copy link
Member

@wiseweb-works bu PR'da deployed hata veriyor, sorun nedir tam bilmiyorum sizin bir bakma şansınız var mıdır sorunun kaynağı nedir?

Diğer PR'larda deployed düzgün çalıştı bu arada.

@wiseweb-works
Copy link
Member

wiseweb-works commented Dec 13, 2024

mesajlasma-uygulamalari.md dosyaları satır 120'de tek seferde yazılabilecek harf sayısı olan 800 aşılmış. Normalde 400 idi bizim birçok dosyamızda sorun oldu diye onu 800 yaptık ama daha da fazlası aşırı oluyor. O satırı bir yerden ikiye bölerseniz sorun çözülür.

Ayrıca markdown table oluştururken --- veya :---: gibi maksimum 3 tire içerir şekilde kullanılması gerekiyor. 3'den fazla kullanılması temelde bize fayda sağlamıyor.

Gene markdown tablosunun başlık kısmında | Başlık | gibi sadece 1'er boşluk olacak şeklide olması daha doğru.

Edit: Tekrar aynı yerde aynı hatayı aldınız. 1200 karakter olarak görünüyor.
https://github.com/GuvendeKal/guvendekal.org/actions/runs/12321351084/job/34394690655?pr=196#step:4:159

/github/workspace/docs/mesajlasma-uygulamalari.md:120:801 MD013/line-length Line length [Expected: 800; Actual: 1122]

@GokturkTalha
Copy link
Member

Edit: Tekrar aynı yerde aynı hatayı aldınız. 1200 karakter olarak görünüyor. https://github.com/GuvendeKal/guvendekal.org/actions/runs/12321351084/job/34394690655?pr=196#step:4:159

/github/workspace/docs/mesajlasma-uygulamalari.md:120:801 MD013/line-length Line length [Expected: 800; Actual: 1122]

Evet, dosya içinde genel bazı hatalarda vardı, düzelttim şuan. Hata çıkartmaması lazım.

@GokturkTalha
Copy link
Member

Haritalar sayfasında da bazı eksik bölümler var ancak onları düzeltiriz, çekiyorum şimdilik. İşleri daha fazla karıştırmamak için.

@GokturkTalha GokturkTalha merged commit d12d674 into GuvendeKal:main Dec 13, 2024
1 check 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.

3 participants