Remove unused dependency on coherent.licensed

[abravalheri]

Summary of changes

With the changes introduced in ae480ff, the dependency on coherent.licensed is unused, so I think it would be good to remove it for clean-up purposes¹.

The approach I took here was to simply comment out the line, because I notice that is the approach previously used in #4975, but I can also simply remove all the requires and put an empty array if that is preferred.

Closes #5002

Pull Request Checklist

• Changes have tests
• News fragment added in newsfragments/.
  (See documentation for details)

Footnotes

1. The removal should speed up build and facilitate for downstream consumers building from git repos instead of sdists.
   I understand that in the long run it would be nice to have a solution for the cycles, but for now because we are not using the dependency, there is not much point in keep it around. ↩

[jaraco]

Thanks Anderson for this. See my note in the issue for why I'm hesitant to simply remove the dependency.

[jaraco]

Let's go ahead and apply this change here (only) for now to alleviate the concern for now.