flat icon indicating copy to clipboard operation
flat copied to clipboard

Node upgrade / deprecation warnings

Open alecglen opened this issue 2 years ago • 19 comments

Getting these in the action summary:

image

alecglen avatar Oct 14 '22 00:10 alecglen

I am meeting this too. I see that the last commit in this repo is in Aug 2021. Is this action still maintained?

akosiaris avatar Dec 16 '22 09:12 akosiaris

I'm also seeing these warnings. Hope it'll be resolve. Node LTS is already on v18 👀

aslak01 avatar Jan 20 '23 13:01 aslak01

Any word on this? are there any plans to patch this action so it does not throw this Node 12 warning and eventually stop working?

skhalaf21 avatar Feb 08 '23 21:02 skhalaf21

@idan , @Wattenberger , @mattrothenberg , Bueller...

alecglen avatar Feb 12 '23 16:02 alecglen

any news ?

NargiT avatar Mar 19 '23 23:03 NargiT

My project no longer runs as of today due to node 12 depreciation. Anyone know of an alternative to this or willing to make a fork?

Edit on closer inspection it appears to be a problem with the deno package repo, but summer of depreciation is coming quite fast.

aslak01 avatar May 23 '23 19:05 aslak01

Very frustrating abandonment and lack of communication here; it really makes me distrustful of other Github Next initiatives...

@aslak01 Personally I switched to using an Azure data factory a while ago. I'm not a Node guy, but it might not be much work to roll my Python into an action with similar functionality. I can take a swing at it sometime in the next month if there's interest?

alecglen avatar May 23 '23 20:05 alecglen

@alecglen @aslak01 I totally hear your frustration. Sincere apologies for the delay on this front.

I have a PR open that should hopefully fix this issue: https://github.com/githubocto/flat/pull/86.

I'm by no means an actions expert, so can you try giving this PR a test by pinning the version of the action in your repo to githubocto/flat@mr/node16 and letting me know if it works?

mattrothenberg avatar May 24 '23 10:05 mattrothenberg

@mattrothenberg

Hi, thanks for your work on this. I have switched to using the action you suggested today. It fixes the issue reported here and also works fine for my use case. I 'll let it run (it runs on a schedule) for a couple of weeks and report back here.

akosiaris avatar Jun 06 '23 07:06 akosiaris

This should hopefully be fixed in https://github.com/githubocto/flat/releases/tag/3.4.0 🤞

mattrothenberg avatar Jun 07 '23 08:06 mattrothenberg

Much appreciated, @mattrothenberg!

I think this is working for me too, but there's something I'm missing. Hopefully the gang assembled here can help make it make sense?

I haven't updated my my flat.yaml since the release, but I checked just now and the deprecation warnings are indeed gone as of a couple weeks ago. I'm confused because from what I can tell in the action logs, it's still running with an older version (3.3.1 based on the SHA).

...
Download action repository 'githubocto/flat@v3' (SHA:fc34373cc20b987e89d75d96435a9fecd8fb0807)
...

However, the warnings are definitely gone, so I'm not sure what to believe :shrug: . Is there anything I need to do explicitly in the yaml to make sure I'm on the new release?

alecglen avatar Jun 21 '23 20:06 alecglen

@alecglen That sounds weird! Are you able to link to the particular action run where you're seeing that?

mattrothenberg avatar Jun 21 '23 21:06 mattrothenberg

Yeah I can ...it's just in a private repo that isn't actually top-secret haha. Just sent you a collaborator invite so you can view it.

alecglen avatar Jun 21 '23 21:06 alecglen

Great, will take a look tomorrow!

mattrothenberg avatar Jun 21 '23 21:06 mattrothenberg

@mattrothenberg

Hi, thanks for your work on this. I have switched to using the action you suggested today. It fixes the issue reported here and also works fine for my use case. I 'll let it run (it runs on a schedule) for a couple of weeks and report back here.

Couple of weeks passed, not a single error for me. Many thanks again!

akosiaris avatar Jun 21 '23 22:06 akosiaris

Still getting this on flat@v3 unfortunately? Not sure why.

dalanmiller avatar Jul 19 '23 05:07 dalanmiller

@dalanmiller have you tried with githubocto/[email protected] ?

skhalaf21 avatar Jul 19 '23 19:07 skhalaf21

Ahhhh wonderful that was it.

I assumed incorrectly that using @v3 would grab the latest minor point release.

dalanmiller avatar Jul 19 '23 23:07 dalanmiller

@dalanmiller no problem. Actually, that's what I also though, but apparently it was not the case.

skhalaf21 avatar Jul 20 '23 16:07 skhalaf21