Skip to content
This repository has been archived by the owner on Nov 24, 2022. It is now read-only.

we need a better way to include("config") #7

Open
ckingbailey opened this issue Oct 16, 2018 · 1 comment
Open

we need a better way to include("config") #7

ckingbailey opened this issue Oct 16, 2018 · 1 comment
Labels
dev interface make programming easier

Comments

@ckingbailey
Copy link
Owner

As of right now, it'd be easy to add it to the autoload path. I believe that goes under PSR-1

Pointing to the config file remains a problem in v2 because of the nested file structure, and will be an issue here if we start to adopt v2. Likewise pointing to 'vendor/autoload.php' is a problem in v2 for the same reason.

Let's use this thread to discuss possible solutions.

@ckingbailey
Copy link
Owner Author

ckingbailey commented Nov 14, 2018

I'm now autoloading it with Composer

This technique seems to be working

Let's keep this issue open for discussion in case autoloading gives us any problems

@ckingbailey ckingbailey added the dev interface make programming easier label Dec 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dev interface make programming easier
Projects
None yet
Development

No branches or pull requests

1 participant