Skip to content

Commit e42fa97

Browse files
authored
Drop pry as default runtime console. (#947)
Using `pry` as the default runtime console in the production environment is forcing us to account for it in Discourse core's codebase. In order to avoid the additional complexity in Discourse core's codebase, we have decided to drop `pry` as the default runtime console and rely on IRB which is just as good these days. We will however be keeping the dependency on `pry` around so those that would like to use it can start it manually.
1 parent 3f9f4fa commit e42fa97

File tree

1 file changed

+1
-7
lines changed

1 file changed

+1
-7
lines changed

templates/web.template.yml

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -240,13 +240,7 @@ run:
240240
chmod: +x
241241
contents: |
242242
#!/bin/bash
243-
# If they requested a console, load pry instead
244-
if [ "$*" == "c" -o "$*" == "console" ]
245-
then
246-
(cd /var/www/discourse && RAILS_ENV=production sudo -H -E -u discourse bundle exec pry -r ./config/environment)
247-
else
248-
(cd /var/www/discourse && RAILS_ENV=production sudo -H -E -u discourse bundle exec script/rails "$@")
249-
fi
243+
(cd /var/www/discourse && RAILS_ENV=production sudo -H -E -u discourse bundle exec script/rails "$@")
250244
251245
- file:
252246
path: /usr/local/bin/rake

0 commit comments

Comments
 (0)