Black doesn't split too long for-in in list comprehension

[AdeliManesh]

Describe` the bug
In my python file a line with more than 80 characters exists but black doesn't split the line into multi lines due the line length constraint.

To Reproduce
the is my "test.py" file.

graph_path_expressions_in_local_constraint_refinements = [
	graph_path_expression
	for refined_constraint in self._local_constraint_refinements.values()
	if refined_constraint is not None
	for graph_path_expression in refined_constraint.condition_as_predicate.variables
]

Run Black:$ black -l 79 --preview test.py
All done! ✨ 🍰 ✨

Expected behavior

This is the result which i get from yapf and it is ok for me.

graph_path_expressions_in_local_constraint_refinements = [
	graph_path_expression for refined_constraint in
		self._local_constraint_refinements.values()
		if refined_constraint is not None for graph_path_expression in
		refined_constraint.condition_as_predicate.variables
]

but black doesn't make any changes and gives the below result:

graph_path_expressions_in_local_constraint_refinements = [
	graph_path_expression
	for refined_constraint in self._local_constraint_refinements.values()
	if refined_constraint is not None
	for graph_path_expression in refined_constraint.condition_as_predicate.variables
]

Environment

• Black's version: black, 22.12.0 (compiled: yes)
• OS and Python version: Ubuntu 18.04/Python 3.8.16

[felix-hilden]

Thanks for submitting this! Here's a simplified case on the playground.

[Pathfinder216]

This is also a problem with dictionary comprehensions. Here's another playground case showing both.

[sataset]

coming from #4590. @JelleZijlstra thank you for mentioning an existing issue

i see that it is 2 years old now. will this issue be addressed any time in future?

[JelleZijlstra]

I have no plans to work on it but if a contributor makes a PR fixing this, I will review it.

There are a number of similar issues in the linetoolong label; in various, mostly fairly unusual cases Black does not format code to fit within the line limit. Personally, in a codebase formatted with Black, I would simply turn off any lint rules that complain about formatting-related issues like line length; Black does a good job and it's not worth my time to manually fix the few remaining issues.