Add ComposerMustBeValid and ComposerLockMustBeFresh insights

[Jibbarth]

Q	A
Bug fix?	no
New feature?	yes
Fixed tickets	#...

Add new Insights about composer.

Example:

• [Architecture] Composer.json is not valid:
  composer.json: The property name is required
  composer.json: The property description is required
  composer.json: No license specified, it is recommended to do so. For closed-source software you may use "proprietary" as license.

• [Architecture] The lock file is not up to date with the latest changes in composer.json:
  You may be getting outdated dependencies. Run update to update them.

[olivernybroe]

Really nice changes! Would it be too much work to add blacklist of different composer package names?
This way we can show that they have to change the package name away from laravel/laravel in the Laravel preset :) (else this could be another pr)

[Jibbarth]

@olivernybroe I think this feature is already handled by https://github.com/nunomaduro/phpinsights/blob/master/src/Domain/Insights/ComposerMustContainName.php ?

[olivernybroe]

Oh yeah, totally forgot, sorry! :)

[olivernybroe]

Looks great, sound like a nice addition. Could you add a few tests for this also?

[Jibbarth]

Could you add a few tests for this also?

I'll try 😉

[Jibbarth]

@olivernybroe Tests added. Is it good for you ?

[olivernybroe]

Looks great! The tests are nice.
I'll say, just go ahead and merge it in :)

[Jibbarth]

Thanks a lot for the review @olivernybroe 👍