Only infer module operations on string Const nodes

[DanielNoord]

Steps

• For new features or bug fixes, add a ChangeLog entry describing what your PR does.
• Write a good description on what the PR does.

Description

Closes #1700

Type of Changes

	Type
✓	🐛 Bug fix

[coveralls]

Pull Request Test Coverage Report for Build 2644730710

• 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.02%) to 92.187%

---

Totals
Change from base Build 2644346545:	0.02%
Covered Lines:	9498
Relevant Lines:	10303

---

💛 - Coveralls

[Pierre-Sassoulas]

Should we release 2.12.2 right now ?

[jacobtylerwalls]

Let me get a clean run of both primer suites locally. I should have done that yesterday.

[DanielNoord]

Do we need this for the upgrade to 2.12.x? If it's just a crash fix we might want to wait to see we can get 2.12.1 to work with pylint 2.15.x.

[jacobtylerwalls]

It was found in the old pylint primer and was caused in astroid 2.12.0, so it's necessary if we want all the checks to pass when upgrading to 2.12.0-1. But if you mean something like "wait and make sure this is the only blocker" i.e. want to wait for the first report from Pierre on his sys.path patching PR that's fine too.

[DanielNoord]

It was found in the old pylint primer and was caused in astroid 2.12.0, so it's necessary if we want all the checks to pass when upgrading to 2.12.0-1. But if you mean something like "wait and make sure this is the only blocker" i.e. want to wait for the first report from Pierre on his sys.path patching PR that's fine too.

Yeah. I'd say let's make sure we get all fixes necessary for the 2.12.x upgrade into 2.12.2. Nothing inherently bad with releasing a number of versions, but it only costs us more time.