-
Notifications
You must be signed in to change notification settings - Fork 46
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
Error when setting action version via SHA #67
Comments
Is this possibly caused by aa0f798 & Co? Why does your action have fancy code to find its own revision? That's curious / interesting - doesn't GitHub check-out the actions? Or is this not failing to clone the action, but the repo? The "source" or the "target" |
Was this change I made in 8e058c4 the culprit causing rossjrw/pr-preview-action#67?! Signed-off-by: Michael Vorburger <[email protected]>
Apparently the change I made in www-learn-study/saraswati.learn.study@8e058c4 was the culprit causing this?! Reverting that in www-learn-study/saraswati.learn.study#55 seems to do the trick (but makes www-learn-study/saraswati.learn.study#50 re-appear, but that's another story). That using a verion tag instead of a SHA fixes this seems really weird, to me... no action should prevent me from using a SHA (which is considered more secure) instead of a Tag! |
It's to display the specific action version in the message with the preview link. E.g. if you use
You're right, and in fact, this was already fixed in #36. But since then, I've managed to accidentally remove that fallback in aa0f798. This one's on me. |
Fixed in v1.4.6 |
In https://github.com/www-learn-study/saraswati.learn.study/actions/runs/6729181929/job/18289629833?pr=20 I ran into this:
The text was updated successfully, but these errors were encountered: