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

docs - updates Linkedin example to follow new tested pattern #546

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jtroussard
Copy link
Contributor

Updating the Linkedin example so that it follows the new pattern provided by #530

@jtroussard jtroussard requested a review from JonathanHuot April 21, 2024 18:23
@jtroussard jtroussard added this to the 2.0.1 milestone Apr 21, 2024
@jtroussard
Copy link
Contributor Author

@JonathanHuot might need some guidance on the scope of the unit test, also maintainer access to the vault might be more granular, however I have not tried to push anything to secrets using the cli yet.

@coveralls
Copy link

Coverage Status

coverage: 90.097%. remained the same
when pulling cf7420d on docs-examples-linkedin
into 77905a4 on master.

@jtroussard jtroussard marked this pull request as draft April 21, 2024 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants