-
Notifications
You must be signed in to change notification settings - Fork 32
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
شکستن رویدادها در یک فایل به ازای یک روز #90
Comments
من رو این کار میکنم و روندش رو اینجا میگذارم |
به نظرم فعلا بهتره دست نگه داری. با وجود #35 و یه تغییر بزرگ، اینکار خیلی فرسایشی میشه. میشه وقتی که اون مرج شد اینو انجامش داد |
شکستن به رویداد چک کردن برای ورودی تکراری رو خیلی سخت میکنه٬ شاید بهتر باشه به روز شکسته بشه |
موافقم. اسم گذاری فایل هم ساده تر میشه
…On Tue, Oct 8, 2019, 9:48 PM Kian Ostad ***@***.***> wrote:
شکستن به رویداد چک کردن برای ورودی تکراری رو خیلی سخت میکنه٬ شاید بهتر
باشه به روز شکسته بشه
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#90?email_source=notifications&email_token=AAHEJ4YVKYRWQVGHMO37XPDQNTPZ3A5CNFSM4I5Y2RBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAVMF7Q#issuecomment-539673342>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAHEJ43RUFOPQSEQQB53L5TQNTPZ3ANCNFSM4I5Y2RBA>
.
|
من دارم رو این کار میکنم و الان سوال شد که ساختار پوشه و فایل چطور باشه؟
or
یا چیز دیگه؟ |
اولی بهتره به نظرم. سیصد و شصت و پنج تا فایل تو یه پوشه منطقی نیست
…On Mon, Oct 14, 2019, 11:10 PM Kian Ostad ***@***.***> wrote:
من دارم رو این کار میکنم و الان سوال شد که ساختار پوشه و فایل چطور باشه؟
jalali/01-farvadin/13.yaml
or
jalali/0113.yaml
یا چیز دیگه؟
@fzerorubigd <https://github.com/fzerorubigd> @hramezani
<https://github.com/hramezani>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#90?email_source=notifications&email_token=AAHEJ45PM5MGZUPCSH5BWQLQOTN6DA5CNFSM4I5Y2RBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBGRU7Y#issuecomment-541923967>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAHEJ474PWUBLBY5F2NIHATQOTN6DANCNFSM4I5Y2RBA>
.
|
یک مورد دیگه اینکه این رو دارم رو کامند |
اوکی٬ البته تعدادش خیلی کمتر از ۳۶۵ تا میشه. شاید ۵۰ تا هم نشه |
اون split صرفا یه کامند گذرا بود که رویدادهای یه سال رو بریزم تو قالب
ماهانه. الانم میتونی همونو کلا تغییر بدی، جایی استفاده نمیشه
…On Mon, Oct 14, 2019, 11:18 PM Kian Ostad ***@***.***> wrote:
یک مورد دیگه اینکه این رو دارم رو کامند
split
پیاده میکنم و یک پارامتر بهش اضافه کردم به نام
-base
و مقدارش میتونه
month (current code) or event (this issue)
باشه. فکر کردم شاید بد نباشه بعدا براساس نیاز مثلا براساس تقویم هم بشه
خروجی گرفت
@fzerorubigd <https://github.com/fzerorubigd>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#90?email_source=notifications&email_token=AAHEJ42ALNOW6NQNXFYG7ADQOTO3HA5CNFSM4I5Y2RBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBGSSQQ#issuecomment-541927746>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAHEJ44LBNEV5C3U5OXQCM3QOTO3HANCNFSM4I5Y2RBA>
.
|
با وضعیت الان میتونه هر مدلی ورودی بگیره (حتی از خروجی خودش) و به هر مدلی خروجی بده |
در مورد تعداد، باید در نظر داشته باشی که با ساختار فعلی مثلا میشه تولد و مرگ هم اضافه کرد، تو دو تا تقویم جدا. احتمالا با تعداد زیادی رو برو بشیم |
پیشنهاد من این است که بعد از اینکه سه بخش اصلی تمام شد - جلالی ، قمری و میلادی - رویداد ها را با کمک یک ابزار تفکیک کنیم.
معایب :
مزایا:
The text was updated successfully, but these errors were encountered: