Skip to content

Conversation

jakubkoci
Copy link
Contributor

The initial goal with samples was to represent API and how to use the framework. However, the content of the folder is more about e2e tests so we can rename it. It doesn't change anything on the fact that it still could (and should) be a good place where the user can see examples of how to use it.

@jakubkoci jakubkoci requested a review from a team May 20, 2021 07:36
jakubkoci added 2 commits May 21, 2021 09:42
Signed-off-by: Jakub Koci <jakub.koci@gmail.com>
Signed-off-by: Jakub Koci <jakub.koci@gmail.com>
@codecov-commenter
Copy link

Codecov Report

Merging #284 (122b9a1) into main (799b6c8) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #284   +/-   ##
=======================================
  Coverage   88.47%   88.47%           
=======================================
  Files         217      217           
  Lines        3897     3897           
  Branches      431      446   +15     
=======================================
  Hits         3448     3448           
  Misses        449      449           

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 799b6c8...122b9a1. Read the comment docs.

@TimoGlastra TimoGlastra merged commit 7bbf7eb into openwallet-foundation:main May 21, 2021
@jakubkoci jakubkoci deleted the refactor/rename-samples branch June 24, 2021 09:35
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