Warn on older versions of func

[gauron99]

This is a subsequent PR to #2

• warn on func version when version is <= 3 than latest (latest - 3 and older)
• one "practical" test in workflow for simplicity (can check manually in the workflow which uses custom version)

please note

• when someone specifies the custom binary url base we skip the version check (dont warn)
• when someone specifies 'latest' version we skip the version check (dont warn)

will add in subsequent PR

• some js tests to cover this ( I want to remove the node_modules first -- use ncc module )

[twoGiants]

Good job! 👍

I think we can make it a bit simpler. Wdyt?

See my comments below.

[twoGiants]

Looks good, looks good! Your JavaScript muscles are getting bigger 💪 😄 , love it!

I left a few comments and would do another round. Hope it's ok! We're close to the finish line 🎸 🎶 🎶

Let me know if anything is unclear.

By the way => next PR should be unit test coverage. Logic is getting more and more complex.

[twoGiants]

Ok, looks good, but lets clean it up a bit and then we're done. I hope it's ok for you to go another round 🙈 .

I would avoid working with null values if I can. Especially in JavaScript. Being explicit makes the code easier to reason about, at least for me 😸. JavaScript is a permissive language 😆 it has undefined, null, NaN as empty/null values and they behave differently 😆. Unlike in go where nil checks are common place => in JavaScript I would use them only when there is no other way. So in the JS code you write, where you can set non-null values => do it and check against that.

[twoGiants]

Looks great! 😸 👍

Thanks for keeping it clean and readable. 🥇