Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Node upgrade / deprecation warnings #76

Open
alecglen opened this issue Oct 14, 2022 · 19 comments
Open

Node upgrade / deprecation warnings #76

alecglen opened this issue Oct 14, 2022 · 19 comments

Comments

@alecglen
Copy link

Getting these in the action summary:

image

@akosiaris
Copy link

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

@aslak01
Copy link

aslak01 commented Jan 20, 2023

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

@skhalaf21
Copy link

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?

@alecglen
Copy link
Author

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

@NargiT
Copy link

NargiT commented Mar 19, 2023

any news ?

@aslak01
Copy link

aslak01 commented May 23, 2023

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.

@alecglen
Copy link
Author

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?

@mattrothenberg
Copy link
Contributor

mattrothenberg commented May 24, 2023

@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: #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?

akosiaris added a commit to akosiaris/greek-oil-refinery-prices that referenced this issue Jun 6, 2023
@akosiaris
Copy link

@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.

@mattrothenberg
Copy link
Contributor

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

@alecglen
Copy link
Author

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 🤷 . Is there anything I need to do explicitly in the yaml to make sure I'm on the new release?

@mattrothenberg
Copy link
Contributor

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

@alecglen
Copy link
Author

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.

@mattrothenberg
Copy link
Contributor

Great, will take a look tomorrow!

@akosiaris
Copy link

@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!

@dalanmiller
Copy link

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

@skhalaf21
Copy link

@dalanmiller have you tried with githubocto/flat@v3.4.0 ?

@dalanmiller
Copy link

Ahhhh wonderful that was it.

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

@skhalaf21
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants