Skip to content

Conversation

@mschoettle
Copy link
Contributor

Throw an error when the max length of the body is exceeded.

In order for the execution to stop and not continue with the remaining merge requests it is necessary to break here:

} catch (err) {
console.error(
'Could not create pull request: !' + mr.iid + ' - ' + mr.title
);
console.error(err);
}

If this is desired, I can add it.

Fixes #208

@spruce
Copy link
Member

spruce commented Jan 29, 2025

Yeah, I think it would be better to stop the migration in that place.

(Later when it is possible to restart a migration it would be even more useful to do so)

@mschoettle
Copy link
Contributor Author

@spruce I added the change that stops the execution when migrating a merge request fails.

@spruce spruce merged commit 0a82136 into piceaTech:master Feb 3, 2025
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.

Issue/PR creation fails if body exceeds max length

2 participants