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.

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

Conduit Documentation #1275

Merged
merged 5 commits into from
Oct 20, 2022
Merged

Conduit Documentation #1275

merged 5 commits into from
Oct 20, 2022

Conversation

winder
Copy link
Contributor

@winder winder commented Oct 13, 2022

Summary

Took a stab at updating the conduit documentation. Some parts are still a little sparse, but I think there is a bit of coverage for everything.

Test Plan

NA

@winder winder self-assigned this Oct 13, 2022
@winder winder marked this pull request as ready for review October 13, 2022 20:18
@codecov
Copy link

codecov bot commented Oct 13, 2022

Codecov Report

Merging #1275 (b644da0) into conduit (6a6fd87) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           conduit    #1275   +/-   ##
========================================
  Coverage    60.12%   60.12%           
========================================
  Files           74       74           
  Lines         9917     9917           
========================================
  Hits          5963     5963           
  Misses        3436     3436           
  Partials       518      518           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@@ -0,0 +1,44 @@
# Creating A Plugin
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we want to talk about what the "lifecycle" of a plugin is and where each function is expected to call?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated. This page probably should be updated to have a separate section for each interface. For now I think the basics are covered.

docs/conduit/plugins/home.md Outdated Show resolved Hide resolved
@winder winder merged commit d0aef01 into algorand:conduit Oct 20, 2022
@winder winder deleted the will/conduit-docs branch October 20, 2022 14:22
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.

3 participants