Skip to content

feat: Make GenerateSourcePluginDocs struct method#124

Merged
yevgenypats merged 5 commits intomainfrom
feat/make_doc_gen_struct_method
Sep 18, 2022
Merged

feat: Make GenerateSourcePluginDocs struct method#124
yevgenypats merged 5 commits intomainfrom
feat/make_doc_gen_struct_method

Conversation

@yevgenypats
Copy link
Copy Markdown
Contributor

Also, generate README.md with all tables
in addition to file per table.

Summary


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 ✅

Also, generate README.md with all tables
in addition to file per table.
@yevgenypats yevgenypats force-pushed the feat/make_doc_gen_struct_method branch from 56d2a12 to 7a3bad2 Compare September 18, 2022 18:16
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.

Looks good 🚀

Can we delete the commented out code?

yevgenypats and others added 4 commits September 18, 2022 22:27
Co-authored-by: Erez Rokah <erezrokah@users.noreply.github.com>
Co-authored-by: Erez Rokah <erezrokah@users.noreply.github.com>
Co-authored-by: Erez Rokah <erezrokah@users.noreply.github.com>
@yevgenypats yevgenypats merged commit 6597df7 into main Sep 18, 2022
@yevgenypats yevgenypats deleted the feat/make_doc_gen_struct_method branch September 18, 2022 19:30
kodiakhq bot pushed a commit that referenced this pull request Sep 19, 2022
🤖 I have created a release *beep* *boop*
---


## [0.6.4](v0.6.3...v0.6.4) (2022-09-18)


### Features

* Make GenerateSourcePluginDocs struct method ([#124](#124)) ([6597df7](6597df7))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
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.

2 participants