Skip to content

Conversation

@HirotoShioi
Copy link
Contributor

@HirotoShioi HirotoShioi commented May 8, 2018

Description

Use Options.Applicative for parsing. Something similar to this

Log classifier

Usage: <interactive> COMMAND [--version]
  Client for peforming analysis on Zendesk

Available options:
  -h,--help                Show this help text
  --version                Show version

Available commands:
  collect-emails           Collect emails requested by single user
  process-tickets          Process all the tickets i.e add comments, tags.
  process-ticket           Process Zendesk ticket of an given ticket id
  raw-request              Raw request to the given url
  show-stats               Print list of ticket Ids that agent has been assigned

Linked Issues

https://iohk.myjetbrains.com/youtrack/issue/CO-281

Notes

I opted to implement as commands instead of arguments since runZendeskMain seemed like it's expecting command instead of argument.

I did try to implement like the example but for some reason, --help command doesn't work.

@HirotoShioi HirotoShioi added the WIP label May 8, 2018
@HirotoShioi HirotoShioi added enhancement New feature or request and removed WIP labels May 9, 2018
@HirotoShioi HirotoShioi self-assigned this May 9, 2018
@HirotoShioi HirotoShioi force-pushed the co-281-fix-argument-parsing branch from 5d99e28 to 7ae3b0a Compare May 9, 2018 04:41
@HirotoShioi HirotoShioi force-pushed the co-281-fix-argument-parsing branch from 7ae3b0a to 6b83383 Compare May 9, 2018 04:45
@HirotoShioi HirotoShioi force-pushed the co-281-fix-argument-parsing branch from 4cc3df9 to 428afc0 Compare May 14, 2018 00:00
@HirotoShioi HirotoShioi force-pushed the co-281-fix-argument-parsing branch from 428afc0 to f5b7a4c Compare May 14, 2018 00:03
@ksaric ksaric merged commit 5d9387e into master May 15, 2018
@ksaric ksaric deleted the co-281-fix-argument-parsing branch May 15, 2018 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants