Closed ABrain7710 closed 3 months ago
@ABrain7710 : What's the status of this one with regards to testing? Should we merge it now that we have done a release.
This is complete, but I haven't had a chance to test it yet. It is a bit of a difficult test.It is a very small change though so if it was reviewed throughly it could be probably be merged
Description
Overall
pull_request_files
,pull_request_commits
, andpull_request_reviews
when they are recollectedImplementation Details
full_collection
boolean flag sostart_data_collection
knows what to passAugurTaskRoutine.start_data_collection
iterate through the tuples and pass therepo_git
andfull_collection
flag to each phasefull_collection
flag to every phase's method arguments. This had to be done sinceAugurTaskRoutine.start_data_collection
passes the same arguments for every phaseget_secondary_data_last_collected
and get prs that have been updated since a datepull_request_files
to only get pr numbers for updated prs iffull_collection
flag is falsepull_request_commits
to only get pr urls for updated prs iffull_collection
flag is falsepull_request_reviews
to only get pr numbers for updated prs iffull_collection
flag is falseNotes for Reviewers I have not tested this yet, I will change it from a draft pr to a pr when it is test
Signed commits