Skip to content
This repository has been archived by the owner on Nov 15, 2022. It is now read-only.

New Kedge structure = new documentation #569

Merged
merged 1 commit into from
Jan 19, 2018

Conversation

cdrage
Copy link
Collaborator

@cdrage cdrage commented Jan 15, 2018

This reorganizes and rewrites many parts of the file reference to
reflect the new format of Kedge (separate keys for each controller).

Tables are now correctly formatted as well as some minor reoganizational
/ structural changes to the documentation.

@cdrage
Copy link
Collaborator Author

cdrage commented Jan 15, 2018

Ping @kadel (this is the last TODO from your PR #565 )

Copy link
Member

@kadel kadel left a comment

Choose a reason for hiding this comment

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

Just one small issue with links, otherwise it looks good

| name | string | yes | The name of the app or microservice this particular file defines. |
| appversion | string | no | The version of the app or microservice this particular file defines. |
| labels | object | no | Map of string keys and values that can be used to organize and categorize (scope and select) objects. May match selectors of replication controllers and services. |
| deployments | array of [deploymentObject](#deploymentObject) | no | [deploymentObject](#deploymentObject) |
Copy link
Member

Choose a reason for hiding this comment

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

those links are not working, it should be lowercase

[deploymentObject](#deploymentobject)

The same for jobs and dpeloymentconfigs

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah... I always forget that 👎 updated!

This reorganizes and rewrites many parts of the file reference to
reflect the new format of Kedge (separate keys for each controller).

Tables are now correctly formatted as well as some minor reoganizational
/ structural changes to the documentation.
@cdrage
Copy link
Collaborator Author

cdrage commented Jan 18, 2018

@kadel I seem to always forget that links don't work unless it's lowercase in markdown. Changes pushed.

@kadel
Copy link
Member

kadel commented Jan 19, 2018

👍 nice work @cdrage

@kadel kadel merged commit e4355b2 into kedgeproject:master Jan 19, 2018
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