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

cli: clarify program source flag description #5571

Merged
merged 3 commits into from
Jul 21, 2023
Merged

cli: clarify program source flag description #5571

merged 3 commits into from
Jul 21, 2023

Conversation

kehiy
Copy link
Contributor

@kehiy kehiy commented Jul 16, 2023

Summary

hello Algorand team, I add a new desc to the clerk CLI help command to show the -F and -P flags should point to a file.

Issues

@kehiy
Copy link
Contributor Author

kehiy commented Jul 16, 2023

@zlangley @winder

@codecov
Copy link

codecov bot commented Jul 16, 2023

Codecov Report

Merging #5571 (594ac15) into master (d316914) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #5571      +/-   ##
==========================================
- Coverage   55.79%   55.77%   -0.02%     
==========================================
  Files         446      446              
  Lines       63418    63418              
==========================================
- Hits        35381    35372       -9     
- Misses      25668    25671       +3     
- Partials     2369     2375       +6     
Impacted Files Coverage Δ
cmd/goal/clerk.go 8.99% <100.00%> (ø)

... and 14 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

cmd/goal/clerk.go Outdated Show resolved Hide resolved
kehiy and others added 2 commits July 21, 2023 17:16
Co-authored-by: Will Winder <wwinder.unh@gmail.com>
@kehiy
Copy link
Contributor Author

kehiy commented Jul 21, 2023

@winder I fixed this, you can merge it now

@winder winder requested a review from a team July 21, 2023 11:59
@winder
Copy link
Contributor

winder commented Jul 21, 2023

Thanks!

@winder winder changed the title fix: cli help cli: clarify program source flag description Jul 21, 2023
@jannotti jannotti merged commit 1a46ceb into algorand:master Jul 21, 2023
@kehiy kehiy deleted the cliDoc branch July 21, 2023 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants