Timing out after succesful publish

[wKovacs64]

Hello, and thanks for your efforts on this project - very cool.

I just tried setting this up and for some reason it's timing out after publishing, but only for the pull_request event (the push event from the same commit succeeds). I set this up on another repo previously and it's not happening there.

Here's the PR:

wKovacs64/pwned#279

Any idea what would be causing this?

(Note: I added a timeout after I noticed it running forever so it would fail faster, but you can see a run I manually cancelled after almost 7 minutes here.)

[Aslemammad]

Thank you so much for the kind words!

It's getting published but the process does not exit, right?

Can we try removing --yes flag from npx?

[wKovacs64]

It's getting published but the process does not exit, right?

That's my understanding, yep.

Can we try removing --yes flag from npx?

Done. Still hangs after publishing, though. 😕

[Aslemammad]

I'll have a look, thank you for letting me know 🙌

[Aslemammad]

I just tried it on my fork and it's working with no issues!

https://github.com/Aslemammad/pwned/actions/runs/10142824058/job/28043119102#step:7:1

what do you think?

Can you try removing the --compact flag too?

[wKovacs64]

I just tried it on my fork and it's working with no issues!

https://github.com/Aslemammad/pwned/actions/runs/10142824058/job/28043119102#step:7:1

what do you think?

Your test is invalid, as the step does exit successfully on push events in my repo, too. It's only pull_request events that hang. So try opening a PR on your fork I guess.

Can you try removing the --compact flag too?

Done, no change (still hangs).

[Aslemammad]

Ohhh you're right! I forgot that case, I'll try again!

[Aslemammad]

Still, it succeed! So I could not reproduce it.

Can you try this command from #166?

API_URL=https://pkg.pr.new npx pkg-pr-new@https://773145bf.stackblitz-cr.pages.dev/stackblitz-labs/pkg.pr.new/pkg-pr-new@166 publish

What this does is it just exits with process.exit when it finishes processing.

let me know.

[wKovacs64]

That worked! I was able to add --yes and --compact back in, too (as expected). 👍

[Aslemammad]

I appreciate that, let me merge it then!