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

Remove cpp examples from readme #72

Merged
merged 3 commits into from
Aug 19, 2024
Merged

Conversation

Corallo
Copy link
Contributor

@Corallo Corallo commented May 2, 2024

Describe your changes

Removing deprecated cpp examples from README.md

@Corallo Corallo requested review from a team as code owners May 2, 2024 11:46
README.md Outdated
for this API exist yet.
[Python][parameter-api-python].
- Video capture API - Enables capture of images from a camera.
There are usage examples available for the Parameter API written in
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you can remove this line - we don't list every example. Alternatively add it under the Examples section instead

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do you mean the line:
"There are usage examples available for the Parameter API written in [Python][opencv-qr-decoder-python]." ?

Copy link
Contributor

Choose a reason for hiding this comment

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

yes

Copy link
Contributor

@madelen-at-work madelen-at-work left a comment

Choose a reason for hiding this comment

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

Looks good but please check the line on the Parameter API example

@madelen-at-work madelen-at-work merged commit d9c2ffb into main Aug 19, 2024
1 of 4 checks passed
@madelen-at-work madelen-at-work deleted the doc/remove-cpp-examples branch August 19, 2024 06:58
johan-olsson-work pushed a commit that referenced this pull request Aug 29, 2024
* Remove cpp examples from readme

* clean-up empty line

* Update README.md

---------

Co-authored-by: Madelen Andersson <madelen.andersson@axis.com>
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