Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Error: Could not update the JSON file: Could not update file: Changes must be made through a pull request." #131

Open
laurent22 opened this issue Feb 13, 2023 · 9 comments
Assignees
Labels
bug Something isn't working

Comments

@laurent22
Copy link

URL to the linked Repository or Organization

laurent22/joplin#7755

Steps to reproduce the problem

There's not specific steps, but for some reason one of the signatures is not being processed:

https://github.com/laurent22/joplin/actions/runs/4155636396

Run contributor-assistant/[email protected]
  with:
    path-to-signatures: readme/cla_signatures.json
    path-to-document: https://github.com/laurent22/joplin/blob/dev/readme/cla.md
    branch: dev
    allowlist: joplinbot,renovate[bot]
    use-dco-flag: false
    lock-pullrequest-aftermerge: true
  env:
    GITHUB_TOKEN: ***
    PERSONAL_ACCESS_TOKEN: ***
CLA Assistant GitHub Action bot has started the process
Error: Could not update the JSON file: Could not update file: Changes must be made through a pull request.

I don't quite understand what that means - the pull request has been signed here like all other pull requests, but for some reason this one in particular does not work. I've also retried the job without success.

Any idea what might be the issue?

@laurent22 laurent22 added the bug Something isn't working label Feb 13, 2023
@laurent22
Copy link
Author

In fact it seems it's now happening with all pull requests. This one too, which was signed but the action failed with same error, and signature file was not updated:

laurent22/joplin#7757

@himself65
Copy link

same error here

@laurent22
Copy link
Author

I don't know if that can help you @himself65, but in our case it was because we changed the protection settings of the branch, and in particular it was required to change the branch via a pull request (and of course the bot doesn't create a pull request, so the changes to the .json file would fail).

@himself65
Copy link

I don't know if that can help you @himself65, but in our case it was because we changed the protection settings of the branch, and in particular it was required to change the branch via a pull request (and of course the bot doesn't create a pull request, so the changes to the .json file would fail).

That makes sense to me. I believe we have the same settings on branch protection.

@himself65
Copy link

Thanks for helping me on this

@ibakshay
Copy link
Member

Hi @himself65, @laurent22,
I have done a major refactoring and did a new release v2.3.0 and unfortunately introduced some bugs in it as well. I will work on it today and release a new version. Till then, please fallback to the previous stable version.
I am really sorry for the inconvenience caused.

uses: contributor-assistant/[email protected]

@laurent22
Copy link
Author

@ibakshay, no problem and thank you for getting back to us. I guess it's more a limitation of the bot than an actual bug. The main problem was that the error message wasn't very clear, which means it wasn't obvious what needed to be done.

@ibakshay
Copy link
Member

@ibakshay, no problem and thank you for getting back to us. I guess it's more a limitation of the bot than an actual bug. The main problem was that the error message wasn't very clear, which means it wasn't obvious what needed to be done.

I have totally mixed things up. I thought you are facing issues after you have upgraded the version to v2.3.0. Thanks a lot for your pull request. I really appreciate it 🎉

@ibakshay
Copy link
Member

ibakshay commented Feb 16, 2023

Thanks for helping me on this

@himself65 Did you also start facing issues after you have configured the branch protection?
This PR #133 from @laurent22 will provide more info on the error message :)

@ibakshay ibakshay self-assigned this Feb 16, 2023
@ibakshay ibakshay moved this to In Progress in Backlog board Feb 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: In Progress
Development

No branches or pull requests

3 participants