8000 fixes CodeIgniter.gitignore deleting a critical framework file by LoganTann · Pull Request #3785 · github/gitignore · GitHub
[go: up one dir, main page]

Skip to content

fixes CodeIgniter.gitignore deleting a critical framework file#3785

Merged
bdougie merged 1 commit intogithub:mainfrom
LoganTann:patch-1
Dec 11, 2021
Merged

fixes CodeIgniter.gitignore deleting a critical framework file#3785
bdougie merged 1 commit intogithub:mainfrom
LoganTann:patch-1

Conversation

@LoganTann
Copy link
Contributor
  • Provide a link to the application or project’s homepage: https://github.com/codeigniter4/CodeIgniter4
  • Provide links to documentation supporting the change you’re making : https://codeigniter.com/user_guide/concepts/structure.html#system

    System : This directory stores the files that make up the framework, itself. While you have a lot of flexibility in how you use the application directory, the files in the system directory should never be modified.

  • Please consider the scope of your change : ok
  • Please only modify one template per pull request : ok

Explain why you’re making a change :

After spending a lot of time finding why my app won't run on a clean install of my repository, I just noticed that the gitignore I copied introduced a deletion/ignore rule of an internal folder used by the framework.
The current gitignore file deletes the folder /system/Cache/* and causes the following error when running the App from the CLI or the web server : Class "CodeIgniter\Cache\CacheFactory" not found ; making the app unable to run.

I added the following rule to fix this : !system/Cache/*.

Note : this won't fix the bug if the system folder is renamed

After spending a lot of time finding why my app won't run, just noticed that the gitignore I copied introduced a deletion/ignore rule of an internal folder used by the framework.
The current gitignore file deletes the folder `/system/Cache/*` and causes the following error when running the App from the CLI or the web server : `Class "CodeIgniter\Cache\CacheFactory" not found `; making the app unable to run.

I added the following rule to fix this : `!system/Cache/*`.

Note : this won't fix the bug if the system folder is renamed
@LoganTann
Copy link
Contributor Author

(any updates ? this gitignore causes serious problems and need to be updated !)

8000

@bdougie
Copy link
Contributor
bdougie commented Dec 11, 2021

Sorry for the delay. Thanks for the details provided in the OP.

@bdougie bdougie merged commit 9a4b4e3 into github:main Dec 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0