Skip to content

Conversation

joepavitt
Copy link
Contributor

@joepavitt joepavitt commented May 2, 2023

Description

Failed action again, don't believe we need this now? Worth noting we did successfully publish to npm as part of that last action though: https://www.npmjs.com/package/@flowforge/node-red-function-gpt

Checklist

this step is not required for this repo
@joepavitt joepavitt requested a review from knolleary May 2, 2023 11:38
Copy link
Member

@knolleary knolleary left a comment

Choose a reason for hiding this comment

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

It failed as it expected the maintenance branch to exist.

This is part of our standard release process for other repos (although not fully rolled out) - so that maintenance releases happen from the maintenance branch.

However, for these nodes, I think we're more likely to publish from main with any updates (if ever), rather than keeping separate maintenance and development streams.

@knolleary knolleary merged commit c0af046 into main May 2, 2023
@ZJvandeWeg ZJvandeWeg deleted the remove-sync-maintenence branch May 2, 2023 14:16
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 this pull request may close these issues.

2 participants