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

Add support for CSV content from stdin in CLI #637

Merged
merged 1 commit into from
May 25, 2022

Conversation

promulo
Copy link
Contributor

@promulo promulo commented May 24, 2022

Closes #354

This PR enhances the gem's CLI by allowing CSV content to be passed from the standard input.

@promulo promulo marked this pull request as ready for review May 24, 2022 15:53
@promulo promulo removed the request for review from adrianna-chang-shopify May 24, 2022 15:53
@promulo promulo force-pushed the paulo/gh-354/cli-csv-from-stdin branch 2 times, most recently from c8b09fd to db81e1e Compare May 24, 2022 16:43
@promulo promulo marked this pull request as draft May 24, 2022 16:59
@promulo promulo force-pushed the paulo/gh-354/cli-csv-from-stdin branch from db81e1e to fdc83a2 Compare May 25, 2022 09:48
@promulo promulo marked this pull request as ready for review May 25, 2022 11:20
test/lib/maintenance_tasks/cli_test.rb Outdated Show resolved Hide resolved
lib/maintenance_tasks/cli.rb Outdated Show resolved Hide resolved
@etiennebarrie etiennebarrie force-pushed the paulo/gh-354/cli-csv-from-stdin branch from fdc83a2 to b05afb6 Compare May 25, 2022 12:56
@promulo promulo force-pushed the paulo/gh-354/cli-csv-from-stdin branch from b05afb6 to b367bd6 Compare May 25, 2022 13:03
lib/maintenance_tasks/cli.rb Outdated Show resolved Hide resolved
lib/maintenance_tasks/cli.rb Outdated Show resolved Hide resolved
lib/maintenance_tasks/cli.rb Outdated Show resolved Hide resolved
@promulo promulo force-pushed the paulo/gh-354/cli-csv-from-stdin branch from 2c32426 to f59c7c9 Compare May 25, 2022 14:00
@promulo promulo merged commit 415bb0e into main May 25, 2022
@promulo promulo deleted the paulo/gh-354/cli-csv-from-stdin branch May 25, 2022 14:06
lawrencewong pushed a commit to lawrencewong/maintenance_tasks that referenced this pull request Apr 29, 2023
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.

Support running CSV Task from CLI using TTY
3 participants