wwt / SwiftCurrent

A library for managing complex workflows in Swift
https://wwt.github.io/SwiftCurrent/
Apache License 2.0
307 stars 19 forks source link

Docc #148

Closed Tyler-Keith-Thompson closed 2 years ago

Tyler-Keith-Thompson commented 2 years ago

Linked Issue:

Checklist:

codecov-commenter commented 2 years ago

Codecov Report

Merging #148 (32e2544) into main (7ac2343) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #148   +/-   ##
=======================================
  Coverage   91.27%   91.27%           
=======================================
  Files          92       94    +2     
  Lines        2373     2373           
=======================================
  Hits         2166     2166           
  Misses        207      207           
Impacted Files Coverage Δ
Sources/SwiftCurrent/Models/Workflow.swift 100.00% <ø> (ø)
Sources/SwiftCurrent/TypeErased/AnyWorkflow.swift 81.13% <ø> (-1.33%) :arrow_down:
Sources/SwiftCurrent/Models/PassedArgs.swift 100.00% <100.00%> (ø)
...urces/SwiftCurrent/Models/ThenProceedMethods.swift 100.00% <100.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 2c21eb8...32e2544. Read the comment docs.

morganzellers commented 2 years ago

Commit aec111dd58e520ed519faded70272938ccc3c822 seemed to bork the pod spec lint... not sure why

Tyler-Keith-Thompson commented 2 years ago

This was an interesting idea. It was an attempt to use DocC as a way of onboarding contributors to the library. I'm not sure that's a great use-case though. This can remain in our history if we ever want it back, but the PR isn't the biggest priority.

Tyler-Keith-Thompson commented 2 years ago

This was an interesting idea. It was an attempt to use DocC as a way of onboarding contributors to the library. I'm not sure that's a great use-case though. This can remain in our history if we ever want it back, but the PR isn't the biggest priority.