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

Autogenerate documentation for new mock object #5502

Merged
merged 5 commits into from
Jul 14, 2020

Conversation

christian-bromann
Copy link
Member

@christian-bromann christian-bromann commented Jun 26, 2020

Similar how we do this already for browser and element commands we should make sure to auto generate the docs for the commands of the mock object. It should be handled similar to how we do this for call where the actual implementation is somewhere else and the files only contain the docstring.

Result:
Screenshot 2020-06-26 at 03 23 19

Screenshot 2020-06-26 at 03 23 55

@christian-bromann christian-bromann self-assigned this Jun 16, 2020
@christian-bromann christian-bromann changed the base branch from master to cb-network-mock-wait-commands June 26, 2020 01:22
@christian-bromann christian-bromann force-pushed the cb-network-docs branch 2 times, most recently from 9628610 to da9390a Compare June 26, 2020 12:15
@christian-bromann christian-bromann added the needs-review These Pull Requests require review from project members label Jul 9, 2020
@christian-bromann christian-bromann removed the needs-review These Pull Requests require review from project members label Jul 13, 2020
christian-bromann and others added 2 commits July 14, 2020 18:14
Co-authored-by: Mykola Grybyk <25589559+mgrybyk@users.noreply.github.com>
@christian-bromann christian-bromann merged commit 6dabf18 into cb-network-mock-wait-commands Jul 14, 2020
@christian-bromann christian-bromann deleted the cb-network-docs branch July 14, 2020 21:41
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