Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

adding a better help message to lucky exec #1602

Merged
merged 1 commit into from
Nov 3, 2021
Merged

adding a better help message to lucky exec #1602

merged 1 commit into from
Nov 3, 2021

Conversation

jwoertink
Copy link
Member

Purpose

Fixes #1559

Description

This help message didn't tell you about possible options before.

Screen Shot 2021-11-01 at 3 24 52 PM

Checklist

  • - An issue already exists detailing the issue/or feature request that this PR fixes
  • - All specs are formatted with crystal tool format spec src
  • - Inline documentation has been added and/or updated
  • - Lucky builds on docker with ./script/setup
  • - All builds and specs pass on docker with ./script/test

@jwoertink jwoertink merged commit 21ccda6 into master Nov 3, 2021
@jwoertink jwoertink deleted the issues/1559 branch November 3, 2021 18:08
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@matthewmcgarvey matthewmcgarvey

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add the help_message method to lucky exec task
2 participants
@jwoertink @matthewmcgarvey