-
Notifications
You must be signed in to change notification settings - Fork 61
Intelligent guesses for interpolations #110
Comments
Yep sounds valid. And I also feel that before it could've been argued that guessing was bad as there are always edge cases, but now that we give the user the interpolation tags I think guessing is good as if we guess incorrectly the user can simply use the tags and all is good :). I'm about to go back to uni and have other things as well, so I think for the near future at least I wouldn't personally take this one on though, just an FYI. @mxstbr are you thinking of this as something to include in 1.0 or make it a 2.0? Since this would be a breaking change as it would change the way we guess. |
No worries 😉 Whenever you find a minute, you know where to find us! Definitely post-v1, I want to get that out. This isn't like urgent, happy to make it a SemVer-major change. |
I just submitted issue #115 that may be related to this. |
Implemented above heuristic guessing by #219. Not 100% accurate, but should be better than doing nothing. |
Uh oh!
There was an error while loading. Please reload this page.
While the interpolation tagging will already fix most (all?) the edge cases people had with interpolations, there's still a couple of low-hanging fruits where we should have heuristics around what an interpolation likely is.
I think we should be able to do those guesses before we parse for interpolation tags (?), which means we could replace them with the relevant tags and then let the tagging system do it's thing ™️ avoiding a bunch of duplicate logic.
I'm thinking cases like this:
The text was updated successfully, but these errors were encountered: