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

update documentation #9

Merged
merged 19 commits into from
Aug 9, 2024
Merged

update documentation #9

merged 19 commits into from
Aug 9, 2024

Conversation

anjastrunk
Copy link
Contributor

@anjastrunk anjastrunk commented Aug 6, 2024

Update documentation according to #4 (comment).

closes to #4

Signed-off-by: Anja Strunk <[email protected]>
@anjastrunk anjastrunk linked an issue Aug 6, 2024 that may be closed by this pull request
Signed-off-by: Anja Strunk <[email protected]>
Signed-off-by: Anja Strunk <[email protected]>
Signed-off-by: Anja Strunk <[email protected]>
Signed-off-by: Anja Strunk <[email protected]>
Signed-off-by: Anja Strunk <[email protected]>
Signed-off-by: Anja Strunk <[email protected]>
Signed-off-by: Anja Strunk <[email protected]>
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
@markus-hentsch
Copy link
Contributor

Sorry since you added me as a reviewer, I started reviewing before you pushed the latest changes, so some things might not be applicable anymore. I will check again.

anjastrunk and others added 6 commits August 6, 2024 14:54
Co-authored-by: Markus Hentsch <[email protected]>
Signed-off-by: anjastrunk <[email protected]>
Co-authored-by: Markus Hentsch <[email protected]>
Signed-off-by: anjastrunk <[email protected]>
Signed-off-by: anjastrunk <[email protected]>
Co-authored-by: Markus Hentsch <[email protected]>
Signed-off-by: anjastrunk <[email protected]>
Co-authored-by: Markus Hentsch <[email protected]>
Signed-off-by: anjastrunk <[email protected]>
Co-authored-by: Markus Hentsch <[email protected]>
Signed-off-by: anjastrunk <[email protected]>
@anjastrunk
Copy link
Contributor Author

anjastrunk commented Aug 7, 2024

Sorry since you added me as a reviewer, I started reviewing before you pushed the latest changes, so some things might not be applicable anymore. I will check again.

Don't worry, my fault. :-)

@anjastrunk
Copy link
Contributor Author

@markus-hentsch Thanks for your precise review as well as your suggestions. I update README. Please have a second look.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
anjastrunk and others added 2 commits August 9, 2024 07:18
Co-authored-by: Markus Hentsch <[email protected]>
Signed-off-by: anjastrunk <[email protected]>
Co-authored-by: Markus Hentsch <[email protected]>
Signed-off-by: anjastrunk <[email protected]>
Co-authored-by: Markus Hentsch <[email protected]>
Signed-off-by: anjastrunk <[email protected]>
Copy link

github-actions bot commented Aug 9, 2024

Code Coverage

Package Line Rate Health
creator 92%
tests 99%
Summary 96% (217 / 225)

Minimum allowed line rate is 90%

Copy link
Contributor

@markus-hentsch markus-hentsch left a comment

Choose a reason for hiding this comment

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

Thanks for adjusting. LGTM now.

@anjastrunk anjastrunk merged commit c561946 into main Aug 9, 2024
4 checks passed
@anjastrunk anjastrunk deleted the improve-doc branch August 9, 2024 08:09
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.

Generated DID not accepted by Gaia-X Compliance API
2 participants