Add release notes note if no release notes provided

[IanButterworth]

Untested, but should look something like this:

---

@JuliaRegistrator register()

---

Registration pull request created: JuliaRegistries/General/xxxx

Tip: Release Notes

Did you know you can add release notes too? Just add markdown formatted text underneath the comment and it will be added to the registry PR, and if tagbot is installed it will also be added to the tag created on this repository.

Tagging

After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.

This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:

git tag -a v1.3.1 -m "<description of version>" 7d4c8c4c918148afc371e4814d1f2929c3f46b37
git push origin v1.3.1

[ericphanson]

I think this isn't enough; you need to format it to pass this regex:

Registrator.jl/src/commentbot/param_types.jl

Line 49 in 1558fb7

notes_match = match(r"(?:patch|release) notes:(.*)"si, get_body(evt.payload))

like described here: https://github.com/JuliaRegistries/Registrator.jl#release-notes.

But maybe we could relax that regex, or just mention they should write "Release notes:"

[IanButterworth]

Ah ok. I updated it. If JuliaRegistrator writes @JuliaRe... as an example, will that cause the universe to collapse?

[DilumAluthge]

If you put the ping inside of backticks, JuliaRegistrator shouldn't actually get pinged, so you should be good there.

[DilumAluthge]

But if the ping isn't in backticks, I actually do think that the bot will get caught in a loop replying to itself 😂

And eventually hit its rate limit, causing all registrations to grind to a halt.

[ericphanson]

uhhh maybe 🤔. I think in

Registrator.jl/src/commentbot/CommentBot.jl

Line 188 in 1558fb7

function comment_handler(event::WebhookEvent, phrase::RegexMatch)

we could bail if the user is JuliaRegistrator. Or maybe here:

Registrator.jl/src/commentbot/param_types.jl

Line 20 in 1558fb7

user = get_user_login(evt.payload)

which is called by the constructor in the function above. Not totally sure though

[DilumAluthge]

Yeah I was just thinking that. I'll try to make a PR.

[DilumAluthge]

#361

[IanButterworth]

Now that TagBot is fixed can we add this?
See also https://discourse.julialang.org/t/make-breaking-change-releases-easier-to-handle-add-release-notes-to-tags-via-juliaregistrator/103241