Skip to content
This repository has been archived by the owner on May 26, 2023. It is now read-only.

docs(snap): Move usage instructions to docs #440

Merged
merged 1 commit into from
May 6, 2022

Conversation

farshidtz
Copy link
Member

Usage instructions are being moved to docs; see edgexfoundry/edgex-docs#751.
The snap/README has been updated to include development docs only.

Depends on:

Signed-off-by: Farshid Tavakolizadeh [email protected]

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/app-service-configurable/blob/main/.github/CONTRIBUTING.md

PR Checklist

Please check if your PR fulfills the following requirements:

  • I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
  • I am not introducing a new dependency (add notes below if you are)
  • I have added unit tests for the new feature or bug fix (if not, why?)
  • I have fully tested (add details below) this the new feature or bug fix (if not, why?)
  • I have opened a PR for the related docs change (if not, why?)

Testing Instructions

New Dependency Instructions (If applicable)

Signed-off-by: Farshid Tavakolizadeh <[email protected]>
@MonicaisHer MonicaisHer self-requested a review May 4, 2022 13:28
Copy link
Contributor

@MonicaisHer MonicaisHer left a comment

Choose a reason for hiding this comment

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

@farshidtz Thank you, looks good.

@MonicaisHer
Copy link
Contributor

The README doesn't have the "Packaging" section.

@farshidtz
Copy link
Member Author

The README doesn't have the "Packaging" section.

There is an installation section with details. Snap is the only packaging available, as far as I know.

@MonicaisHer MonicaisHer marked this pull request as ready for review May 6, 2022 13:15
@MonicaisHer MonicaisHer merged commit 081a76a into edgexfoundry:main May 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants