Update todo-to-list github action to commit to PR instead of master branch

[shobhitagarwal1612]

Instead of merging the commit to master branch, it commits to the PR's source branch when the PR is approved.

@gino-m PTAL?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 62.22%. Comparing base (94c1ea7) to head (9cf4ae1).
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##             master    #2943   +/-   ##
=========================================
  Coverage     62.22%   62.22%           
  Complexity     1205     1205           
=========================================
  Files           267      267           
  Lines          6380     6380           
  Branches        882      882           
=========================================
  Hits           3970     3970           
  Misses         1863     1863           
  Partials        547      547           

[gino-m]

@shobhitagarwal1612 Should we move this to draft for now?

[shobhitagarwal1612]

@gino-m The workflow isn't able to find the email if the workflow is run after PR is approved automation.

[gino-m]

@gino-m The workflow isn't able to find the email if the workflow is run after PR is approved automation.

Which email are you referring to?

[shobhitagarwal1612]

In order to bypass cla/google check, I added a step to get the email from github.head_ref and use that when committing the new changes from workflow. This works fine when the workflow is executed on commit push. When the workflow is triggered on PR approval, then the step to get email doesn't work.

[gino-m]

In order to bypass cla/google check, I added a step to get the email from github.head_ref and use that when committing the new changes from workflow. This works fine when the workflow is executed on commit push. When the workflow is triggered on PR approval, then the step to get email doesn't work.

I see. Only maintainers can push to this repo, so perhaps allowing creation of TODOs on commit is acceptable?

[shobhitagarwal1612]

I see. Only maintainers can push to this repo, so perhaps allowing creation of TODOs on commit is acceptable?

Agreed. If you also agree, I can update this PR to run the workflow on each commit push.

[gino-m]

I see. Only maintainers can push to this repo, so perhaps allowing creation of TODOs on commit is acceptable?

Agreed. If you also agree, I can update this PR to run the workflow on each commit push.

Sure, let's do it!