Fix variables issue in #3183

[PredictiveManish]

Description
This PR resolves the warnings given in changes in #3183. Removes the unused import sphinx_rtd_theme and import variables __copyright__, __release__, __version__ correctly from metadata.py into conf.py as suggested by reviewbot.

This PR fixes small import issues.

Signed commits

• Yes, I signed my commits.

[MoralCode]

Might also be worth looping in the author of the prior PR that this update is for @JohnStrunk.

Overall i dont really have much opinion on this other than these diffs seem reasonably small - will probably defer explicit approval to other maintainers who have more experience to better evaluate whether these changes are helpful

[PredictiveManish]

Might also be worth looping in the author of the prior PR that this update is for @JohnStrunk.

Overall i dont really have much opinion on this other than these diffs seem reasonably small - will probably defer explicit approval to other maintainers who have more experience to better evaluate whether these changes are helpful

Yes, this issue is not that much big just a small warning! Main issue is the other part of unable to importing properly! it's just a solution for the warning!

[MoralCode]

seems good to me. I'm largely trusting that the readthedocs build worker is a good indicator of whether this works though.

[sgoggins]

LGTM