Skip to content

feat: fetch commits from pull request - #105

Merged
mikededo merged 6 commits into
mainfrom
feat/fetch_commits_from_pullrequest
Dec 14, 2022
Merged

feat: fetch commits from pull request#105
mikededo merged 6 commits into
mainfrom
feat/fetch_commits_from_pullrequest

Conversation

@manerow

@manerow manerow commented Dec 13, 2022

Copy link
Copy Markdown
Collaborator

Code from the UserOptionController to the ExternalRepository to given a pull request fetch all of its commits. The UseCase is tested.

Closes #81

@manerow manerow linked an issue Dec 13, 2022 that may be closed by this pull request
@manerow manerow self-assigned this Dec 13, 2022
@manerow manerow added this to the 2 - MVP milestone Dec 13, 2022
@manerow manerow added type: feat New feature or request status: code review PR ready to be reviewed labels Dec 13, 2022
Comment thread src/main/resources/db/changelog/changes/000_initial_schema.yaml
Comment thread src/main/java/io/pakland/mdas/githubstats/domain/Commit.java Outdated
@manerow
manerow requested a review from a team December 13, 2022 19:41
@mikededo mikededo changed the title Feat: fetch commits from pull request feat: fetch commits from pull request Dec 14, 2022
Comment thread src/test/java/io/pakland/mdas/githubstats/application/AggregateCommitsTest.java Outdated
Comment thread src/test/java/io/pakland/mdas/githubstats/application/AggregateCommitsTest.java Outdated
Removed the comments that were made regarding the `CommitAggregation`
entity as it is something that should be done in a different way and it
was just keeping outdated code commented.
@mikededo
mikededo merged commit 6bc59fb into main Dec 14, 2022
@mikededo
mikededo deleted the feat/fetch_commits_from_pullrequest branch December 14, 2022 17:43
@mikededo mikededo added status: done PR/Issue completed and removed status: code review PR ready to be reviewed labels Dec 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: done PR/Issue completed type: feat New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: fetchCommitsFromPullRequest use case

2 participants