-
Notifications
You must be signed in to change notification settings - Fork 36
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
Added getting started page #701
Conversation
Codecov ReportBase: 67.92% // Head: 67.92% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## main #701 +/- ##
=======================================
Coverage 67.92% 67.92%
=======================================
Files 114 114
Lines 6619 6619
Branches 1157 1157
=======================================
Hits 4496 4496
Misses 2123 2123
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
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.
I'm afraid the gifs are too small. Mainly the one related to code completion, it's very hard to understand what you type
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.
LGTM
package.json
Outdated
{ | ||
"id": "Create Resources", | ||
"title": "Create new resource", | ||
"description": "The plugin allows creating a new Tekton resource directly from the YAML in the editor and deploying it using the Kubernetes: Create command available from the Kubernetes extension.", |
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.
"description": "The plugin allows creating a new Tekton resource directly from the YAML in the editor and deploying it using the Kubernetes: Create command available from the Kubernetes extension.", | |
"description": "Create a new Tekton resource directly from the YAML in the editor and deploy it using the `Kubernetes: Create` command.", |
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.
done
package.json
Outdated
"title": "Create new resource", | ||
"description": "Create a new Tekton resource directly from the YAML in the editor and deploy it using the `Kubernetes: Create` command.", | ||
"media": { | ||
"image": "https://raw.githubusercontent.com/wiki/redhat-developer/vscode-tekton/images/walkthroughs/create-resource.gif", |
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.
Why are we using the raw location of the images? This should point to the local folder image.
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.
Need to fix this.
Signed-off-by: Mohit Suman <[email protected]>
Signed-off-by: Mohit Suman <[email protected]>
Have fixed it in my commit with the images folder and description changes. |
Fix: #691