Skip to content

fix(codegen)!: Column type for slices#447

Merged
kodiakhq[bot] merged 4 commits intomainfrom
fix/codegen/string-array
Dec 7, 2022
Merged

fix(codegen)!: Column type for slices#447
kodiakhq[bot] merged 4 commits intomainfrom
fix/codegen/string-array

Conversation

@candiduslynx
Copy link
Copy Markdown
Contributor

@candiduslynx candiduslynx commented Nov 30, 2022

Summary

  • []*string will become schema.TypeStringArray
  • []*int(8|16|32|64) will become schema.TypeIntArray

Use the following steps to ensure your PR is ready to be reviewed

  • Read the contribution guidelines 🧑‍🎓
  • Run go fmt to format your code 🖊
  • Lint your changes via golangci-lint run 🚨 (install golangci-lint here)
  • Update or add tests 🧪
  • Ensure the status checks below are successful ✅

BEGIN_COMMIT_OVERRIDE
fix(codegen): Column type for slices
END_COMMIT_OVERRIDE

@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 30, 2022

⏱️ Benchmark results

  • Glob-2 ns/op: 145.4
  • BufferedScanner-2 ns/op: 10.01
  • LogReader-2 ns/op: 30.57

Copy link
Copy Markdown
Member

@erezrokah erezrokah left a comment

Choose a reason for hiding this comment

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

This looks good, added a non blocking comment.
Please merge this as fix(codegen)! as this is a breaking change

@candiduslynx candiduslynx changed the title fix(codegen): Column type for slices fix(codegen)!: Column type for slices Dec 1, 2022
@github-actions github-actions bot added breaking and removed fix labels Dec 1, 2022
candiduslynx and others added 4 commits December 7, 2022 10:45
Co-authored-by: Erez Rokah <erezrokah@users.noreply.github.com>
Co-authored-by: Erez Rokah <erezrokah@users.noreply.github.com>
@candiduslynx candiduslynx force-pushed the fix/codegen/string-array branch from 6dc685e to 9fc7fe0 Compare December 7, 2022 07:45
@kodiakhq kodiakhq bot merged commit 7474c90 into main Dec 7, 2022
@kodiakhq kodiakhq bot deleted the fix/codegen/string-array branch December 7, 2022 07:50
erezrokah added a commit to erezrokah/plugin-sdk that referenced this pull request Dec 7, 2022
@github-actions github-actions bot added breaking and removed fix labels Dec 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants