Code review comment for lp:~tony-badwolf/quickly/editors

Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Looks mostly good as well, thanks for your work there as well :)

Just a question on:

179 + editor = get_editor()
in read_input()

The idea was that quickly configure will still run gedit to have an easy editor for configuring that as well, and not nano or vim. Can't we call there get_quickly_editor() as it was before?

review: Needs Information

« Back to merge proposal