Resolve type aliases in docblocks

[CHItA]

Issue: #23

[CHItA]

@williamdes the aliases from the context don't seem to be used elsewhere, so the later changes should work fine, and not break anyone's builds. I also added unit tests to prevent regression. I consider this PR finished and ready to be merged. If you find anything let me know.

[williamdes]

This looks good, I will have another time a look onto your work

[williamdes]

Closing to try to trigger the CIs

[CHItA]

As far as I know

pull_request:
        branches:
            - main

should be enough, there is no need to match the name of my branch.

[williamdes]

As far as I know

pull_request:
        branches:
            - main

should be enough, there is no need to match the name of my branch.

is that to specify the PR target ?

[CHItA]

Yup.

[codecov]

Codecov Report

Merging #24 (1724f16) into main (76dd2c8) will increase coverage by 0.23%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main      #24      +/-   ##
============================================
+ Coverage     55.19%   55.43%   +0.23%     
  Complexity     1017     1017              
============================================
  Files            50       50              
  Lines          2607     2612       +5     
============================================
+ Hits           1439     1448       +9     
+ Misses         1168     1164       -4     

Impacted Files	Coverage Δ	Complexity Δ
src/Parser/DocBlockParser.php	90.69% <100.00%> (+0.69%)	28.00 <7.00> (+1.00)
src/Parser/NodeVisitor.php	48.13% <100.00%> (+0.32%)	126.00 <0.00> (-1.00)	⬆️
src/Parser/ParserContext.php	63.33% <0.00%> (+6.66%)	26.00% <0.00%> (ø%)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 76dd2c8...1724f16. Read the comment docs.

[williamdes]

Yup.

Awesome, thank you so much !

All tests pass except php 7.1
https://github.com/code-lts/doctum/pull/24/checks?check_run_id=1664642199#step:7:14

Would you too agree to drop support for PHP 7.1 ?

[CHItA]

It is fine by me. Not sure why this would fail though.

[williamdes]

It is fine by me. Not sure why this would fail though.

It fails because they have a version that is PHP <= 7.1 compatible and one that is compatible PHP > 7.1
Many hacks for some support..
Anyway, I will do more work on this before merging your work
Thank you for your help, this seems to be perfect

[williamdes]

💯

[williamdes]

I released 5.4.0-dev phar, you can try it out and let me know ;)

[CHItA]

Seems to work great, thanks.