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

ENDOC-522 new edits & Engr comments #549

Merged
merged 3 commits into from
Aug 15, 2022
Merged

ENDOC-522 new edits & Engr comments #549

merged 3 commits into from
Aug 15, 2022

Conversation

jyunmitch
Copy link
Collaborator

No description provided.

@jyunmitch jyunmitch requested review from nshaw and Lyd1aCla1r3 August 12, 2022 21:21
vuepress/docs/next/docs/getting-started/ent-bundle.md Outdated Show resolved Hide resolved
vuepress/docs/next/docs/getting-started/ent-bundle.md Outdated Show resolved Hide resolved
vuepress/docs/next/docs/getting-started/ent-bundle.md Outdated Show resolved Hide resolved
vuepress/docs/next/docs/getting-started/ent-bundle.md Outdated Show resolved Hide resolved
|`ent ecr generate-cr`|Generate the custom resource |
|`ent ecr get-bundle-id [repository-url]` | Calculate and display the bundle ID
|`ent ecr get-plugin-id --auto [repository-url]` | Calculate and display the plugin ID
|`ent ecr get-plugin-code --auto --repo=[repository-url]` |Calculate and display the plugin code for a Git-based v1 bundle|
|`ent ecr get-plugin-code [component-name] --repo=[repository-url]` | Calculate and display the plugin code for a Docker-based v5 bundle
Copy link
Contributor

Choose a reason for hiding this comment

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

same as above; docker-based and no v5

Copy link
Member

Choose a reason for hiding this comment

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

Agree, just remove v5 from the note.

@@ -163,4 +165,4 @@ These ent CLI commands are required when using a Git-based (v1) bundle.
ent appname YOUR-APPNAME
ent namespace YOUR-NAMESPACE
```
Check out [Run Blueprint-generated Microservices and Micro Frontends in Dev Mode](../../tutorials/create/ms/run-local.md) for additional information.
Follow the [Build and Publish a Project Bundle](../../tutorials/create/pb/publish-project-bundle.md) tutorial for more details about bundling your next Docker-based project.
Copy link
Contributor

Choose a reason for hiding this comment

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

awkward/confusing wording. and can apply to creating bundles for your current project, not just what you do next

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

will when the actual tutorial has a new title so will change with that

vuepress/docs/next/docs/getting-started/ent-diag.md Outdated Show resolved Hide resolved
vuepress/docs/next/docs/getting-started/ent-diag.md Outdated Show resolved Hide resolved
@jyunmitch jyunmitch merged commit 12759e1 into main Aug 15, 2022
@jyunmitch jyunmitch deleted the ENDOC-522 branch August 15, 2022 19:51
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