Skip to content
This repository has been archived by the owner on Oct 2, 2024. It is now read-only.

Features/missing ref warning instead of error #2

Conversation

Rasmus-Rosted
Copy link

@Rasmus-Rosted Rasmus-Rosted commented Jan 21, 2021

This implementation gives the user the possibility to decide if the action should fail with an error or continue with a warning. This is done in order to make the action more flexible in the cases where a branch does not exist but the rest of the workflow that the action is used in is not dependent on that condition.

@Rasmus-Rosted Rasmus-Rosted changed the title Warning instead of error when branch is not found features/missing ref warning instead of error Jan 21, 2021
@Rasmus-Rosted Rasmus-Rosted changed the title features/missing ref warning instead of error Features/missing ref warning instead of error Jan 21, 2021
ref: "heads/" + branch
})
} catch (error) {
const shouldFailSoftly = (soft_fail === 'true');

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you not have soft_fail as boolean from the start (not as string) in JS?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I might have misunderstood something, but it does not seem like it from what is described in actions/toolkit#361

Maybe I just haven't looked into it enough

@Rasmus-Rosted Rasmus-Rosted merged commit 28303e9 into features/add-owner-repo-input Jan 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants