MRChemSoft / mrchem

MultiResolution Chemistry
GNU Lesser General Public License v3.0
27 stars 21 forks source link

Fix importing of pyparsing #420

Closed robertodr closed 2 years ago

robertodr commented 2 years ago

If an older pyparsing is available (and it usually always is), then it is imported, but it won't work with the parser generated by parselglossy. I've added a check so that if the existing pyparsing is not of major version >=3, the local copy is used instead. I will port this to parselglossy itself as well.

codecov[bot] commented 2 years ago

Codecov Report

Merging #420 (37295f0) into master (9a55e74) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #420   +/-   ##
=======================================
  Coverage   68.38%   68.38%           
=======================================
  Files         183      183           
  Lines       15448    15448           
=======================================
  Hits        10564    10564           
  Misses       4884     4884           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.