Skip to content
This repository has been archived by the owner on Sep 20, 2022. It is now read-only.

Create ProtocolVisitor #59

Merged
merged 2 commits into from
Jan 29, 2021
Merged

Create ProtocolVisitor #59

merged 2 commits into from
Jan 29, 2021

Conversation

dfed
Copy link
Collaborator

@dfed dfed commented Jan 28, 2021

Creates a ProtocolVisitor in the visitors module. This visitor is capable of determining type information about a protocol.

@codecov
Copy link

codecov bot commented Jan 28, 2021

Codecov Report

Merging #59 (4caaa04) into main (c5d6a6f) will increase coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #59      +/-   ##
==========================================
+ Coverage   99.28%   99.31%   +0.02%     
==========================================
  Files          27       28       +1     
  Lines         842      873      +31     
==========================================
+ Hits          836      867      +31     
  Misses          6        6              
Impacted Files Coverage Δ
...urces/SwiftInspectorVisitors/ProtocolVisitor.swift 100.00% <100.00%> (ø)
Sources/SwiftInspectorVisitors/StructVisitor.swift 100.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 c5d6a6f...4caaa04. Read the comment docs.

@dfed dfed marked this pull request as ready for review January 28, 2021 23:43
@dfed dfed requested a review from fdiaz January 28, 2021 23:43
Copy link
Owner

@fdiaz fdiaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really cool stuff, I'm really liking how this is building on top of each other now 👍

Sources/SwiftInspectorVisitors/ProtocolVisitor.swift Outdated Show resolved Hide resolved
@dfed dfed merged commit 472d283 into main Jan 29, 2021
@dfed dfed deleted the dfed--protocol-visitor branch January 29, 2021 02:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants