Skip to content
This repository has been archived by the owner on Jul 1, 2023. It is now read-only.

composer: use separate autoload and autoload-dev #140

Closed
mfn opened this issue Jul 12, 2019 · 3 comments
Closed

composer: use separate autoload and autoload-dev #140

mfn opened this issue Jul 12, 2019 · 3 comments

Comments

@mfn
Copy link

mfn commented Jul 12, 2019

Currently the whole project tree is marked for autoload, from composer.json

    "autoload": {
        "classmap": [
            "./"
        ]
    },

It does not differentiate between autoloading actual source and test related stuff, but should.

(unverified) suggestion:

    "autoload": {
        "classmap": [
            "./src/"
        ]
    },
    "autoload-dev": {
      "classmap": [
        "./tests/"
      ]
    }  
@jrfnl
Copy link

jrfnl commented Feb 17, 2022

A PR is open to address in php-parallel-lint/PHP-Parallel-Lint#88

@grogy
Copy link
Contributor

grogy commented Feb 17, 2022

PR was merged. Thanks @mfn for suggestion and @jrfnl for pushing as PR 👍

@mfn
Copy link
Author

mfn commented Mar 21, 2022

Nice, closing then!

@mfn mfn closed this as completed Mar 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants