jgorset/facebook-messenger

Rails 6 recommends removing `require_dependency`

Open

#255 geöffnet am 4. Sept. 2019

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Ruby (215 Forks)github user discovery
good first issuehelp wanted

Repository-Metriken

Stars
 (970 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

This gem recommends using require_dependency in the initializer to set up bot reloading in development but those should be removed according to the rails 6 upgrade guide using the Zeitwerk autoloader.

It seems to me that maybe only config.autoload_paths is needed nowadays, so maybe we should update the Readme?

Contributor Guide

Research-Richtung
Überprüfen Sie den Rails 6 Upgrade Leitfaden zum automatischen Laden und aktualisieren Sie die README, indem Sie `require dependency` durch `config.autoload paths` ersetzen.
Tech Stack
ruby
Domain
backend
Issue Type
Dokumentation
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitRuby
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80