Skip to content

fix: typo metadata support to csv parser#10742

Merged
powersj merged 2 commits intoinfluxdata:masterfrom
JackSharebourg:fix-typo-metadata-support-to-csv-parser
Feb 28, 2022
Merged

fix: typo metadata support to csv parser#10742
powersj merged 2 commits intoinfluxdata:masterfrom
JackSharebourg:fix-typo-metadata-support-to-csv-parser

Conversation

@JackSharebourg
Copy link
Copy Markdown
Contributor

Required for all PRs:

  • [N/A] Updated associated README.md.
  • Wrote appropriate unit tests.
  • Pull request title or commits are in conventional commit format

The error msg should refer to the parameter name.

@telegraf-tiger
Copy link
Copy Markdown
Contributor

Thanks so much for the pull request!
🤝 ✒️ Just a reminder that the CLA has not yet been signed, and we'll need it before merging. Please sign the CLA when you get a chance, then post a comment here saying !signed-cla

@telegraf-tiger telegraf-tiger bot added the fix pr to fix corresponding bug label Feb 26, 2022
@JackSharebourg
Copy link
Copy Markdown
Contributor Author

!signed-cla

@telegraf-tiger
Copy link
Copy Markdown
Contributor

@JackSharebourg JackSharebourg marked this pull request as ready for review February 26, 2022 19:30
Copy link
Copy Markdown
Contributor

@Hipska Hipska left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No breaking change since feature wasn't released yet.

@Hipska Hipska added area/csv csv parser/serialiser related plugin/parser 1. Request for new parser plugins 2. Issues/PRs that are related to parser plugins labels Feb 28, 2022
@powersj powersj merged commit c2458fe into influxdata:master Feb 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/csv csv parser/serialiser related fix pr to fix corresponding bug plugin/parser 1. Request for new parser plugins 2. Issues/PRs that are related to parser plugins

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants