Remove unused visit_package function

[DanielNoord]

	Type
✓	🔨 Refactoring

Description

See:

I'm not too familiar with the pyreverse code, but even if it isn't called anymore, we shouldn't remove it here. That should be done in a separate PR.

Originally posted by @cdce8p in #4940 (comment)

[coveralls]

Pull Request Test Coverage Report for Build 1190951887

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.03%) to 93.088%

---

Totals
Change from base Build 1189311027:	0.03%
Covered Lines:	13211
Relevant Lines:	14192

---

💛 - Coveralls

[Pierre-Sassoulas]

@DudeNr33 what do you think ?

[DudeNr33]

It indeed seems like this method is never executed.
I removed it locally and ran pyreverse over the codebase without issues.
When adding some logging to the method and running pyreverse no output appears.
This is probably because the underlying ASTWalker class determines the method to call for the visit of each node by doing a getattr(handler, f"visit_{kid}"), where kid is the lowercase class name of the astroid node.
And since astroid does not have a node class named "Package", this method is never called.

So I say go for it!

As a side note for a potential future cleanup MR, while looking at the code I noticed that the whole tagging logic of the nodes has become obsolete in the Linker class, as we now use fully qualified names in the output files.

[Pierre-Sassoulas]

I'm trusting the current leading expert in pyreverse on that one :)