Compat Helper Spams.

[mattBrzezinski]

No description provided.

[DilumAluthge]

This "keep old compat" versus "drop old compat" business is more trouble than it's worth.

Let's just get rid of the "drop old compat" feature, and only do the "keep old compat" stuff.

This would technically be breaking, so we'll have to release CompatHelper 4.0.

I don't think very many people use the "drop old compat" feature, anyway. And if they do, they can use CompatHelper v2.

[DilumAluthge]

If we always use the "keep old compat" codepath, then we can probably also do what you said on Slack:

Can we not just check if a PR title which contains the dependency and the version we are bumping to exists already? Instead of trying to match the title exactly?

[mattBrzezinski]

Looking into why keep existing compat is being added in here. We can probably save this drop features part by spending some more time looking at it.

https://github.com/davidanthoff/PyRhodium.jl/pulls

[mattBrzezinski]

I cannot understand why the above two PRs;

• CompatHelper: bump compat for "DataFrames" to "1" davidanthoff/PyRhodium.jl#92
• CompatHelper: bump compat for "Distributions" to "0.25" davidanthoff/PyRhodium.jl#91

Were created. By default we are passing in KeepEntry as the action. When we call pr_title_parenthical we should always be appending ", (keep existing compat)". For some reason we're not? The workflow for this repo isn't overriding it either?

😕

[mattBrzezinski]

All of the PRs created for:

• https://github.com/SciML/DataDrivenDiffEq.jl/pulls
• https://github.com/SciML/DiffEqOperators.jl/pulls
• https://github.com/SciML/DiffEqGPU.jl/pulls

Were most likely using v3.0.0 which we have seemingly resolved. Nothing new will be created in those repos, and looking at all of their existing actions, they seem to run hourly and have been fine since picking up the patched version of CompatHelper.

[DilumAluthge]

I cannot understand why the above two PRs;

• CompatHelper: bump compat for "DataFrames" to "1" davidanthoff/PyRhodium.jl#92
• CompatHelper: bump compat for "Distributions" to "0.25" davidanthoff/PyRhodium.jl#91

Were created. By default we are passing in KeepEntry as the action. When we call pr_title_parenthical we should always be appending ", (keep existing compat)". For some reason we're not? The workflow for this repo isn't overriding it either?

😕

Do the actions logs show anything useful?