Skip to content
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

Detect when MR has a description that is too long #212

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

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)

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