Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

chore: refactor ast parser and inspector #24

Merged
merged 6 commits into from
Jul 30, 2024
Merged

chore: refactor ast parser and inspector #24

merged 6 commits into from
Jul 30, 2024

Conversation

g4s8
Copy link
Owner

@g4s8 g4s8 commented May 9, 2024 •
edited

Add new flags with glob patterns for -files and -types, deprecated -type and -all flags.

Add types resolver to get types from different files and packages.

Implement project scan for types.

Now envdoc should be able to resolve documentation for env tags from different files and types.

Close #23
Fix #20
Fix #16
Fix #15

@codecov Codecov
Copy link

codecov bot commented May 9, 2024 •
edited

Codecov Report

Attention: Patch coverage is 44.53961% with 259 lines in your changes missing coverage. Please review.

Project coverage is 43.08%. Comparing base (f8f8c47) to head (591db74).

Files Patch % Lines
config.go 0.00% 110 Missing ⚠️
parser.go 44.15% 32 Missing and 11 partials ⚠️
debug.go 0.00% 40 Missing ⚠️
main.go 0.00% 32 Missing ⚠️
render.go 0.00% 31 Missing ⚠️
converter.go 97.14% 2 Missing and 1 partial ⚠️
Additional details and impacted files 
@@             Coverage Diff             @@
##           master      #24       +/-   ##
===========================================
- Coverage   93.89%   43.08%   -50.82%     
===========================================
  Files           9       10        +1     
  Lines         672      564      -108     
===========================================
- Hits          631      243      -388     
- Misses         34      309      +275     
- Partials        7       12        +5

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

g4s8 added 2 commits May 24, 2024 12:37
@g4s8
chore: refactor ast parser and inspector
ed59a7b 
Keep current context for `ast.File` in AST parser.
Iterate over all files in file-set to traverse with AST parser.

For #23
@g4s8
feat: multi-files structure parsers
5df0879 
Reimplemented the AST logic to parse multiple files from
single file-set and parse types documentation from the fileset.

Ref: #23
@g4s8 g4s8 marked this pull request as ready for review July 30, 2024 08:36
@g4s8 g4s8 self-assigned this Jul 30, 2024
@g4s8
Copy link
Owner Author

g4s8 commented Jul 30, 2024

TODO: needs to improve coverage.

@g4s8 g4s8 mentioned this pull request Jul 30, 2024
Open
3 tasks
@g4s8 g4s8 merged commit 14b5c52 into master Jul 30, 2024
3 of 5 checks passed
@g4s8 g4s8 deleted the i23 branch July 30, 2024 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@g4s8 g4s8

Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@g4s8