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

Added telescope download proto with cli docs #693

Merged
merged 1 commit into from
Nov 13, 2024
Merged

Conversation

NorOldBurden
Copy link
Collaborator

No description provided.


```js
//.protod.config.json example
// `repos` are the repository it's going to clone
Copy link
Collaborator

Choose a reason for hiding this comment

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

there're three parts: owner, repo and branch. branch can be empty and it will use main as default.

```js
//.protod.config.json example
// `repos` are the repository it's going to clone
// `protoDirMapping` is the directory of repo specified if the proto is not under repo's root directory ./protos
Copy link
Collaborator

Choose a reason for hiding this comment

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

This description is not clear. Please be more specified. Thanks!

// `repos` are the repository it's going to clone
// `protoDirMapping` is the directory of repo specified if the proto is not under repo's root directory ./protos
// `outDir` is where the output proto will be put
// `targets` are the target proto to download
Copy link
Collaborator

Choose a reason for hiding this comment

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

This part can also be patterns, please mention that.

@NorOldBurden NorOldBurden force-pushed the update-docs-for-download branch from 5489285 to a6786b6 Compare November 8, 2024 05:41
@NorOldBurden NorOldBurden force-pushed the update-docs-for-download branch from a6786b6 to a835b8d Compare November 13, 2024 05:30
@Zetazzz Zetazzz merged commit 8912e41 into main Nov 13, 2024
2 checks passed
@NorOldBurden NorOldBurden changed the title Added telescope download docs Added telescope download cli docs Nov 14, 2024
@NorOldBurden NorOldBurden changed the title Added telescope download cli docs Added telescope download proto with cli docs Nov 14, 2024
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.

2 participants