You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Looking at ansiblelint.main.path_inject() I see that it does not normalize the paths before comparing them. I think it should, to work in all cases. I.e. I see that the paths set in #1885 (comment) contain '//', and so will not be properly compared. (I've no idea if this matters to issue #1885.)
As long as i'm writing, I suggest main.activate_venv() in place of main.path_inject() because it says both what is happening and why.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
Hello,
Looking at ansiblelint.main.path_inject() I see that it does not normalize the paths before comparing them. I think it should, to work in all cases. I.e. I see that the paths set in #1885 (comment) contain '//', and so will not be properly compared. (I've no idea if this matters to issue #1885.)
As long as i'm writing, I suggest main.activate_venv() in place of main.path_inject() because it says both what is happening and why.
Beta Was this translation helpful? Give feedback.
All reactions