Skip to content

add same-line docs to aggregate types & templates#455

Merged
WebFreak001 merged 1 commit intodlang-community:masterfrom
WebFreak001:agg-type-same-line-comments
Feb 2, 2022
Merged

add same-line docs to aggregate types & templates#455
WebFreak001 merged 1 commit intodlang-community:masterfrom
WebFreak001:agg-type-same-line-comments

Conversation

@WebFreak001
Copy link
Member

Now code like class X {} /// foo properly gets documentation, like in the dmd -D output.

Extends #438

This code in D-Scanner now properly gets recognized as documented:

class C{} /// a
interface I{} /// b
enum e = 0; /// c
void f(){} /// d
struct S{} /// e
template T(){} /// f
union U{} /// g

This will fully fix dlang-community/D-Scanner#499

Now code like `class X {} /// foo` properly gets documentation, like in
the `dmd -D` output.
@codecov
Copy link

codecov bot commented Feb 2, 2022

Codecov Report

Merging #455 (1d3cbd7) into master (beadf0b) will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #455      +/-   ##
==========================================
+ Coverage   82.83%   82.84%   +0.01%     
==========================================
  Files          11       11              
  Lines        8306     8311       +5     
==========================================
+ Hits         6880     6885       +5     
  Misses       1426     1426              
Impacted Files Coverage Δ
src/dparse/parser.d 91.29% <100.00%> (+0.01%) ⬆️

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 beadf0b...1d3cbd7. Read the comment docs.

@WebFreak001 WebFreak001 merged commit 205fa27 into dlang-community:master Feb 2, 2022
@WebFreak001 WebFreak001 deleted the agg-type-same-line-comments branch February 2, 2022 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

It does not recognize documentation on same line

2 participants

Comments