Looking into the tests… What syntax do you prefer?...

# github-workflows-kt

[louiscad]

Looking into the tests…
What syntax do you prefer? (both are working fine now)

expr { env.GITHUB_ACTIONS } shouldBe "$dollar{{ GITHUB_ACTIONS }}"
expr { env.GITHUB_ACTIONS } shouldBe "\${{ GITHUB_ACTIONS }}"

[LeoColman]

abc shouldBe ("$" + "{{ GITHUB_ACTIONS }}")

[louiscad]

Why do you prefer resorting to concatenation when backslash works?

[LeoColman]

This makes it clear that I want a $ before everything, and separating the two parts with a + is easier on my eyes to understand what exactly I'm trying to test

[LeoColman]

In prod code I'll probably go for String template, but for testing - where my goal is to show other humans wtf I'm testing - I prefer to make it a little bit more "explain like I'm 5"

[LeoColman]

My problem with concatenation is that it's ugly, but sometimes (mainly when using $) it can be clearer

[LeoColman]

Reminds me of having to do something like "${"$"}" to escape the dollar xD

[louiscad]

Wouldn't explaining the backslash to a 5 years old work just as well, if not better?

[louiscad]

Also, it's not like we get 5 year old folks contributing 😄

[LeoColman]

They probably all compile down to the exact same thing, right?

[LeoColman]

I think it's up to personal preference. Do you have a particular reason to dislike concatenation? In this particular case it's not that obscene as "www" + website + Constants.SLASH + "path" + variable + ...

[louiscad]

It adds many more characters, which is greater noise IMHO

[LeoColman]

Maybe it's a problem for me. I have a hard time reading \${{}} -- Like having to force my brain to think what it means instead of just 'knowing'

[LeoColman]

And of course this takes less than 2 seconds, so it's purely a style choice

[LeoColman]

What about

abc should be """${{ GITHUB_ACTIONS }}"""

[LeoColman]

It would add a couple more ", but remove the """"ambiguity"""" of escaping the $

[louiscad]

It doesn't work because string templates are allowed in multiline strings as well.

[louiscad]

To me \ means "we're not putting a Kotlin variable or expression in there, it's just a $ character."

[LeoColman]

But ${ isn't a string template, so it won't try to auto-complete in """, right?

[LeoColman]

Maybe I'm remembering it wrong

[louiscad]

Try and you'll see. play.kotl.in

[louiscad]

"""${{ lol }}""" doesn't compile

[louiscad]

Unless lol is defined of course

[LeoColman]

Dammit, ${ is actually a template due to ${foo.bar}

[LeoColman]

I confused it with $(, which would probably compile fine

[louiscad]

Yup, it does