-
Notifications
You must be signed in to change notification settings - Fork 3
Dokku'da yanlış ruby versiyonu kuruluyor #216
Comments
Tek yolu Gemfile'a ruby versiyonu yazmak değil. CI tarafında bu iş .ruby-version dosyasıyla kontrol ediliyor, ayrıca Gemfile'a versiyon eklenirse testler ne olur bilmiyorum (muhtemelen ruby-version kazanır). ruby-version deklarasyonu rubocop tarafından da dikkate alınıyor. Bu durumda "ön tanımlı Ruby versiyonu budur" manasında bir .ruby-version dosyası bana daha iyi bir çözüm gibi geliyor. CI gördüğüm kadarıyla zaten versiyonun üzerine basarak test yapıyor, yani o taraf bundan etkilenmez. |
şu anda https://github.com/omu/nokul/blob/develop/.ruby-version dosyası var ama 2.3.1. kuruyor. |
Yukarıdaki issue'dan dolayı öyle yapıyor. Önerdikleri Gemfile içinde |
@ecylmz şu testleri kastediyorum (yeni eklendi) https://github.com/omu/nokul/blob/develop/.circleci/parallel_workflow.yml |
@msdundar Heroku'nun önerdiği yol şu: https://devcenter.heroku.com/articles/ruby-versions#specifying-a-ruby-version-via-the-environment: Gemfile'da Bu yöntemle |
Son sürümlerde Rubocop Ruby versiyonunu Gemfile.lock'tan alıyor: rubocop/rubocop#5632 |
Buna göre 2 yol:
Her iki yol da: (1) Dokku'yu memnun ediyor, (2) CI testleri çalışıyor, (3) Tooling sorunsuz (en azından Rubocop, diğerlerini bilmiyorum; .ruby-version kullanmak bir parça daha avantajlı olabilir gelecek durumlar için) Benim oyum ilk yönteme gider. Bir dosyadan kurtuluyoruz, daha parametrik bir çözüm. Karar her ne olacaksa buna uygun PR yapılmalı. |
@ecylmz senin için de uygunsa ilk çözüm için (Gemfile) bir PR hazırlar mısın bu issue'yu kapatan? |
bk #219 |
Bunun nedeni, Gemfile'dan ruby versiyonunun kaldırılması. Eğer Gemfile'da ruby versiyonu olmazsa ön tanımlı 2.3.1 kuruluyor. Mevcut deployda sıkıntı yok çünkü daha önce 2.5.1 kullanıldığını biliyor. Bu sorun yalnızca ilk deployda ortaya çıkıyor.
Kısacası Gemfile'a ruby versiyonu yazmak zorundayız.
The text was updated successfully, but these errors were encountered: