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

Fixed broken kdoc links and some other warnings. #232

Merged
merged 1 commit into from
Mar 29, 2019

Conversation

zach-klippenstein
Copy link
Collaborator

@zach-klippenstein zach-klippenstein commented Mar 29, 2019

Also added configuration to generate dokka output for some modules.

Also added configuration to generate dokka output for
some modules.
@zach-klippenstein zach-klippenstein changed the title Zachklipp/fix kdoc Fixed broken kdoc links and some other warnings. Mar 29, 2019
@zach-klippenstein zach-klippenstein requested a review from rjrjr March 29, 2019 01:23
@zach-klippenstein zach-klippenstein marked this pull request as ready for review March 29, 2019 01:23
*
* **Never call [Workflow.compose] directly, always do it through this context method.**
* **Never call [StatefulWorkflow.compose] or [StatelessWorkflow.compose] directly, always do it
* through this context method.**
*
* 1. If the child _wasn't_ already running, it will be started either from
* [initialState][Workflow.initialState] or its snapshot.
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: no longer really an or situation, since we unified the methods.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Those are still related though, and I want to avoid confusion about people trying to call them directly.

@zach-klippenstein zach-klippenstein merged commit 541c054 into master Mar 29, 2019
@zach-klippenstein zach-klippenstein deleted the zachklipp/fix-kdoc branch March 29, 2019 17:13
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.

2 participants