emacs-ess / ESS

Emacs Speaks Statistics: ESS
https://ess.r-project.org/
GNU General Public License v3.0
623 stars 162 forks source link

fix error handling in esslint() #1291

Closed bastistician closed 6 months ago

bastistician commented 6 months ago

lintr::lint() errors were catched but failed to get reported due to a trivial coding error that this PR fixes.

The Flymake log buffer showed

Warning [ess-r-flymake  *ess-r-flymake*]:  Error in cat("@@warning: @@", e) :
  argument 2 (type ’list’) cannot be handled by ’cat’

rather than the catched error message.

Earlier confusion about this broken warning can be found in #883, #1016, #1238. After this patch, I can see and investigate the underlying lint() error, which in my case was "Malformed config file".