phalcon / phalcon-devtools

Phalcon Developer Tools
https://docs.phalcon.io/latest/en/devtools
Other
1.33k stars 630 forks source link

Fix #1508 devtools version showing phalcon version instead for 4.1.x #1513

Closed Ultimater closed 3 years ago

Ultimater commented 3 years ago

Hello!

In raising this pull request, I confirm the following (please check boxes):

Small description of change:

4.1.x uses _getVersion() instead of getVersion() when extending Phacon\Version, this results in devtools reporting back the wrong version of devtools to the user as though devtools and phalcon have the same version when they really have different versions.

Thanks

codecov[bot] commented 3 years ago

Codecov Report

Merging #1513 (5cb8f5a) into 4.1.x (5cf8c9a) will not change coverage. The diff coverage is 0.00%.

@@          Coverage Diff          @@
##           4.1.x   #1513   +/-   ##
=====================================
  Coverage   3.27%   3.27%           
=====================================
  Files        112     112           
  Lines       5122    5122           
=====================================
  Hits         168     168           
  Misses      4954    4954