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 processing multiple input lines #102

Merged
merged 1 commit into from
Aug 22, 2023

Conversation

atc0005
Copy link
Owner

@atc0005 atc0005 commented Aug 22, 2023

  • add support for reading input URLs from a file
  • add support for reading input URLs from stdin
  • retain interactive prompt if no input given
  • remove leading explanation text from results
  • emit URL parsing failure text to stderr, continue processing URLs
    • allow for batch operation
    • allow for splitting success results and error output across different files

refs GH-78

- add support for reading input URLs from a file
- add support for reading input URLs from stdin
- retain interactive prompt if no input given
- remove leading explanation text from results
- emit URL parsing failure text to stderr, continue
  processing URLs
  - allow for batch operation
  - allow for splitting success results and error output
    across different files

refs GH-78
@atc0005 atc0005 added this to the v0.3.0-alpha.1 milestone Aug 22, 2023
@atc0005 atc0005 self-assigned this Aug 22, 2023
@atc0005
Copy link
Owner Author

atc0005 commented Aug 22, 2023

Will plan to resolve the linter warnings in the next alpha release.

@atc0005 atc0005 merged commit d3b5c19 into development Aug 22, 2023
27 of 28 checks passed
@atc0005 atc0005 deleted the i78-process-multiple-input-lines branch August 22, 2023 10:46
atc0005 added a commit that referenced this pull request Aug 22, 2023
Split output and decoding logic to resolve gocognit linting
error.

refs GH-102
refs GH-78
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant