Skip to content

chore: check file permissions on Github Action - #9556

Closed
ddevsr wants to merge 7 commits into
codeigniter4:developfrom
ddevsr:file-permission-check
Closed

chore: check file permissions on Github Action#9556
ddevsr wants to merge 7 commits into
codeigniter4:developfrom
ddevsr:file-permission-check

Conversation

@ddevsr

Copy link
Copy Markdown
Collaborator

Description
See https://github.com/codeigniter4/CodeIgniter4/actions/runs/15002007798/job/42151150139

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@ddevsr
ddevsr requested review from kenjis and paulbalandanMay 14, 2025 02:43
@ddevsrddevsr added the github_actions Pull requests that update Github_actions code label May 14, 2025

@paulbalandanpaulbalandan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is composer update needed if those files were already in ThirdParty/? Maybe dependabot's head branch is not updated with develop?

@ddevsr
ddevsr marked this pull request as draft May 14, 2025 03:33
@ddevsr

Copy link
Copy Markdown
CollaboratorAuthor

Using latest commit db79418 still not get update

@ddevsr

Copy link
Copy Markdown
CollaboratorAuthor

@paulbalandan

Copy link
Copy Markdown
Member

I have tried this locally with rm -rf vendor composer.lock and it really results in the error.

@ddevsrddevsr closed this May 14, 2025
@ddevsr
ddevsr deleted the file-permission-check branch May 14, 2025 16:14
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actionsPull requests that update Github_actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@ddevsr@paulbalandan