Ashutosh Kasudhan

Results 35 comments of 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...