Skip to content

feat(github): collect comments/reviews#2484

Merged
mindlesscloud merged 2 commits into
apache:mainfrom
merico-ai:2452-featuregithub-get-all-normal-comments-review-review-comments-and-store-it-in-tablepull_request_comments
Jul 18, 2022
Merged

feat(github): collect comments/reviews#2484
mindlesscloud merged 2 commits into
apache:mainfrom
merico-ai:2452-featuregithub-get-all-normal-comments-review-review-comments-and-store-it-in-tablepull_request_comments

Conversation

@warren830
Copy link
Copy Markdown
Contributor

Summary

  1. collect normal comments, inline comments and reviews for github
  2. store normal/inline comments in _tool_github_pull_request_comments
  3. store reviews in _tool_github_pull_request_reviews
  4. convert all comments/reviews to pull_request_comments

Does this close any open issues?

closes #2452

Screenshots

_tool_github_pull_request_comments
image
_tool_github_pull_request_reviews
image
pull_request_comments
image

Other Information

Any other information that is important to this PR.

@warren830 warren830 added this to the v0.12.0 milestone Jul 13, 2022
@warren830 warren830 added pr-type/feature-development This PR is to develop a new feature component/plugins This issue or PR relates to plugins labels Jul 13, 2022
@warren830 warren830 force-pushed the 2452-featuregithub-get-all-normal-comments-review-review-comments-and-store-it-in-tablepull_request_comments branch from 384df48 to 8e122bf Compare July 13, 2022 02:08
Comment thread models/migrationscripts/init_schema.go Outdated

func (*initSchemas) Version() uint64 {
return 20220707232344
return 20220711232344
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Isn't this going to cause this migration to be ran again and therefore drop all existing tables via Up()? Wouldn't that cause data loss?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, this will cause data loss

@warren830 warren830 force-pushed the 2452-featuregithub-get-all-normal-comments-review-review-comments-and-store-it-in-tablepull_request_comments branch 10 times, most recently from 840fecd to 3a63db2 Compare July 15, 2022 04:18
@warren830 warren830 force-pushed the 2452-featuregithub-get-all-normal-comments-review-review-comments-and-store-it-in-tablepull_request_comments branch 3 times, most recently from deedf33 to f31e9b4 Compare July 15, 2022 06:12
@warren830 warren830 force-pushed the 2452-featuregithub-get-all-normal-comments-review-review-comments-and-store-it-in-tablepull_request_comments branch from f31e9b4 to b8097ca Compare July 15, 2022 06:15
Copy link
Copy Markdown
Contributor

@mindlesscloud mindlesscloud left a comment

Choose a reason for hiding this comment

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

LGTM

@mindlesscloud mindlesscloud merged commit 3e59c99 into apache:main Jul 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/plugins This issue or PR relates to plugins pr-type/feature-development This PR is to develop a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature][GitHub] Get all normal comments, review, review comments and store them in table.pull_request_comments

3 participants