Skip to content

Library time #125

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

Merged
merged 8 commits into from
Feb 24, 2023
Merged

Library time #125

merged 8 commits into from
Feb 24, 2023

Conversation

serkan-bayram
Copy link
Contributor

No description provided.

@egeakman
Copy link
Collaborator

egeakman commented Feb 22, 2023

Python belgeleri rST formatında yazıldığı için bazı noktalara dikkat etmemiz gerekiyor. Bunlardan bazıları:

  • Back-tick (ters kesme işareti) (`) arasına yazılmış kelimelere ek getirirken `asdasd asdasd`e yerine `asdasd asdasd`'e şeklinde araya kesme işareti koyarak yazmamız gerekiyor.

  • Back-tick'li ifadelerden sonra alttan çizgi geliyorsa bu bir link veya referans olduğunu gösterir bu yüzden örneğin `epoch`_ şeklindeki ifade al çizgi bozulmadan `epoch`_ 'a şeklinde yazılabilir.

  • Back-tick sayısının tutarlı olması gerekiyor. Yani back-tick'li ifade tek bt ile başladıysa tek bt ile bitmesi, çift ise çift ile bitmesi gerekir.

  • Yıldız işaretleri italik veya kalın yazmak için kullanıldığından tekse tek çiftse çift yıldızla kapatılması gerekir. Örneğin *sth yerine *sth*, sth** yerine **sth** gibi.

Bunlarda eksiklik olursa syntax hataları ortaya çıkabiliyor. sphinx-lint aracı bu hataları çıktı olarak veriyor.

@serkan-bayram
Copy link
Contributor Author

Tamamdır bu dediklerinize dikkat ederek gözden geçiriyorum şu anda.

@egeakman
Copy link
Collaborator

egeakman commented Feb 22, 2023

Tamamdır bu dediklerinize dikkat ederek gözden geçiriyorum şu anda.

Bu arada bu tarz hataları (özellikle back-tick ile ilgili olanları) VS Code'un arama kısmında regex kullanarak kolayca bulmak mümkün oluyor. Bazı kullandığım regexler:

:`[^`]+`[a-zA-Z0-9]
 ``[^`]+``[a-zA-Z0-9]+

@serkan-bayram
Copy link
Contributor Author

Bu değişiklikleri nasıl gönderebilirim acaba yukarıda yaptığım gibi bir review uygun oluyor mu yoksa başka bir şey mi yapmam lazım

@egeakman
Copy link
Collaborator

Bu değişiklikleri nasıl gönderebilirim acaba yukarıda yaptığım gibi bir review uygun oluyor mu yoksa başka bir şey mi yapmam lazım

Direkt GitHub üzerinden review edebilirsiniz suggestion'larla veya localde tümden değiştirebilirsiniz. Size nasıl rahat oluyorsa.

@egeakman
Copy link
Collaborator

egeakman commented Feb 22, 2023

Bu çıkan checklerin sonucunda nerelerde hata olduğunu daha rahat yakalayabilirsiniz. Örneğin https://github.com/python/python-docs-tr/actions/runs/4247504160/jobs/7385673983#step:5:571 burada kırmızı olan yerler çıkan hatalar. Buradaki hataların satır numarası entry numaralarına göre oluyor o yüzden örneğin 84. satır diyorsa dosyada,

#: library/time.rst:84

bu şekilde başlayan yere bakmanız gerekiyor.

sphinx-lint sonuçlarında normal satır numarası kullanılıyor, o yüzden 100. satır diyorsa kod editöründe görünen 100. satıra bakabilirsiniz.

@serkan-bayram
Copy link
Contributor Author

Tekrardan düzenledim ve pre-commit'ten geçti ama git kullanmayı çok iyi bilmediğimden dolayı tekrar yollamayı beceremedim buraya ek olarak koysam değiştirebiliyor musunuz?

https://drive.google.com/file/d/1Xw_tKsdkRE1xX8odkby03pNWcgpXiDWb/view?usp=share_link

@egeakman
Copy link
Collaborator

Tekrardan düzenledim ve pre-commit'ten geçti ama git kullanmayı çok iyi bilmediğimden dolayı tekrar yollamayı beceremedim buraya ek olarak koysam değiştirebiliyor musunuz?

https://drive.google.com/file/d/1Xw_tKsdkRE1xX8odkby03pNWcgpXiDWb/view?usp=share_link

Elinize sağlık. git push --force yapabilir misiniz?

serkan-bayram and others added 4 commits February 23, 2023 23:38
Co-authored-by: Ege Akman <egeakmanegeakman@hotmail.com>
Co-authored-by: Ege Akman <egeakmanegeakman@hotmail.com>
Co-authored-by: Ege Akman <egeakmanegeakman@hotmail.com>
Co-authored-by: Ege Akman <egeakmanegeakman@hotmail.com>
Copy link
Collaborator

@egeakman egeakman left a comment

Choose a reason for hiding this comment

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

Elinize sağlık 😊

@egeakman egeakman merged commit e7d2b3f into python:3.11 Feb 24, 2023
@serkan-bayram
Copy link
Contributor Author

Teşekkürler sizin de elinize sağlık :)

@serkan-bayram serkan-bayram deleted the library-time branch February 24, 2023 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

library/time.po üzerinde çalışmak istiyorum.
2 participants