-
Notifications
You must be signed in to change notification settings - Fork 45
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 README.MD to reflect new folder structure #130
base: main
Are you sure you want to change the base?
Conversation
@crissancas please can you review and merge if approved? Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Resource Management (RM) is described as "Information about the resources (compute, storage, networking etc) at a given Edge site"
I suggest that we may qualify that the RM is about management (CRUD) of the edge resources by the application providers for the consumption by edge applications.
Changed resource management description per @gunjald comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed resource management text per @gunjald comment
@crissancas or @gunjald please can you merge so I can delete the branch? |
Made Workload Orchestration a section heading Co-authored-by: Nicola Cadenelli <[email protected]>
🦙 MegaLinter status: ✅ SUCCESS
See detailed report in MegaLinter reports |
Changed "Edge Site" to "Edge Cloud Zone"
Thanks @gunjald @nicolacdnll - Both changes suggested above are in the latest commit, please review for approval. |
typos, changed best to optimal
Changes now made 👍 |
Fixes #131