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

update php version and other dependencies #267

Merged

Conversation

bahramsadin
Copy link
Contributor

سلام
وقتتون بخیر
از اونجایی که دیگه php7 منسوخ شده و همه پکیج‌های معروف php هم به نسخه 8 و بالاتر آپدیت شدن بنظرم بهتره توی این پکیج هم آپیدت انجام بدید و یه major release انجام بشه.

@bahramsadin
Copy link
Contributor Author

سلام
خسته نباشید
مشکل یا دلیل خاصی برای مرج نشدن این PR و عدم آپدیت پکیج هست؟
@khanzadimahdi

@khanzadimahdi
Copy link
Member

سلام خسته نباشید مشکل یا دلیل خاصی برای مرج نشدن این PR و عدم آپدیت پکیج هست؟ @khanzadimahdi

سلام. ممنونم بابت پیامتون. راستش پایپ لاین پاس نشده و به همین خاطر من مرجش نکردم. چون در حال حاضر تست درست حسابی نداریم و اگه پایپ لاین هم پاس نشه یعنی یه جای کار میلنگه احتمالا. شما تست کردین که همه چیز درست باشه؟

@bahramsadin
Copy link
Contributor Author

سلام
اگه منظورتون از تست اجرای تست‌های کد هستش که بله ران شده و اکیه و پیوست شد تصویر.
در رابطه با پایپ لاین هم من با این چیزی که الان گذاشتید تا حالا کار نکردم ولی قطعا به خاطر کانفیگ نسخه php باید باشه که خطا میده و باید اصلاح بشه پایپ لاین و اگه تنظیماتش باید از اینجا آپدیت بشه و نسخه جدید php اضافه بشه کامیت جدید بزنم ؟؟https://github.com/shetabit/multipay/blob/master/.travis.yml

ولی کلا اگه یه major release بزنید و این پایپ لاین رو برداریم و بجاش از خود ci/cd گیت هاب استفاده کنیم که تست بشه PR ها خیلی بهتر میشه
در رابطه با استایل کدها هم https://github.com/laravel/pint خیلی بهتر هست و الان در اکثر پکیج ها از این استفاده میشه و میتونیم ازش استفاده کنیم

Screenshot 2025-01-20 161202

@khanzadimahdi khanzadimahdi merged commit f4c3ba8 into shetabit:master Jan 20, 2025
1 of 2 checks 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.

2 participants