Skip to content

Library time - #125

Merged
egeakman merged 8 commits into
python:3.11from
serkan-bayram:library-time
Feb 24, 2023
Merged

Library time#125
egeakman merged 8 commits into
python:3.11from
serkan-bayram:library-time

Conversation

@serkan-bayram

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread library/time.po Outdated
@egeakman

egeakman commented Feb 22, 2023

Copy link
Copy Markdown
Collaborator

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
Copy Markdown
Contributor Author

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

Comment thread library/time.po Outdated
Comment thread library/time.po Outdated
Comment thread library/time.po Outdated
Comment thread library/time.po Outdated
@egeakman

egeakman commented Feb 22, 2023

Copy link
Copy Markdown
Collaborator

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
Copy Markdown
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
Copy Markdown
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

egeakman commented Feb 22, 2023

Copy link
Copy Markdown
Collaborator

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
Copy Markdown
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
Copy Markdown
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?

Comment thread library/time.po Outdated
Comment thread library/time.po Outdated
Comment thread library/time.po Outdated
Comment thread library/time.po Outdated
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>
Comment thread library/time.po Outdated
Comment thread library/time.po Outdated

@egeakman egeakman left a comment

Copy link
Copy Markdown
Collaborator

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
Copy Markdown
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