Ashutosh Kasudhan
Ashutosh Kasudhan
guys pls share your PR link as well along with errors, I need to see your changes as well on which CI checks are failing.
> Hey [@Ashu463](https://github.com/Ashu463), everyone has mentioned their PRs in this thread. Please go through the issue carefully. As of current status, each CI check is passing on your PR, right...
Hey @omgupta-iitk it seems like backend checks on your PR: https://github.com/oppia/oppia/pull/22311 are also passing. Could you please confirm the same ? For merge queue failure, you need to share your...
Hey @U8NWXD we need to make sure that credentials present in oppiabot repository, must be present here in oppia/oppia repo too. I urge to pls check it once, if not...
Hey @U8NWXD and @seanlip here is the video demo of checking CLA signed over issue or not. Here is the link of CI workflow which I showed in video: https://github.com/Ashu463/oppia-test-repo/actions/runs/14761112461/job/41441771521....
Sorry @jayam04 I forgot to assign you, pls respond to my cmnts. Also, I created CLA check for PR as well in the same workflow, now I'm testing it on...
Hey @jayam04 and @seanlip I found one thing, PRs couldn't access repository secrets and hence this workflow couldn't check CLA for PRs; this is explained well in this [article](https://michaelheap.com/access-secrets-from-forks/#:~:text=GitHub%20prevents%20PRs%20from%20forks,use%20them%20for%20malicious%20purposes.) Now...
Hi @seanlip sorry for the delay, actually I was testing solution on my side before proposing it to you. - How CLA action is handled for PRs till now: Actually...
Hey @seanlip should I move my solution to use base64 string in place of sheets cred JSON?
Thanks @seanlip for confirming, I'll update the solution and share demo in upcoming 48 hrs. Lemme try this github app method once on my local, maybe it wouldn't required to...