search

renovatebot / renovate

Home of the Renovate CLI: Cross-platform Dependency Automation by Mend.io
https://mend.io/renovate
GNU Affero General Public License v3.0
17.59k stars 2.31k forks source link

Cache branch/PR state #10232

Closed rarkins closed 1 year ago

rarkins commented 3 years ago

A branch can be in three main states:

Reasons why a branch doesn't exist:

Reasons why a PR doesn't exist:

States which a PR can be in:

This can also be used for more accurate Dependency Dashboard

rarkins commented 3 years ago

Dashboard mockup idea:

This issue contains a list of Renovate updates and their statuses.

Awaiting Branch Creation

These branches can be created by Renovate on-demand once you click their checkbox below.

Branch Exists But Awaiting PR Creation

Tick any checkbox below to manually force creation of any of the following:

Open

Closed / Ignored

Reference: ☑️ Needs manual approval in dashboard 📆 Awaiting scheduled branch creation time 👶 Package is awaiting stabilityDays completion 🚦 Rate limited 🚀 Automerge enabled

viceice commented 3 years ago

Maybe move the emojis after checkbox for better alignment?

rarkins commented 3 years ago

Mockup showing a mix of both:

This issue contains a list of Renovate updates and their statuses.

Awaiting Branch Creation

These branches can be created by Renovate on-demand once you click their checkbox below.

Branch Exists But Awaiting PR Creation

Tick any checkbox below to manually force creation of any of the following:

Open

Closed / Ignored

Reference: ☑️ Needs manual approval in dashboard 📆 Awaiting scheduled branch creation time 👶 Package is awaiting stabilityDays completion 🚦 Rate limited 🚀 Automerge enabled

rarkins commented 3 years ago

Reasons why I'm not sure to do that:

  1. "tags" normally come after the information it's tagging, not before
  2. We might have situations with more than one tag/emoji anyway
viceice commented 3 years ago

🤔 is it possible to use checkboxes in tables? so we use tables for alignment?

rarkins commented 3 years ago

It's not possible inside tables, last time I checked

HonkingGoose commented 3 years ago

🤔 is it possible to use checkboxes in tables? so we use tables for alignment?

It's not possible inside tables, last time I checked

It's still not working (at least not on GitHub).

Example table with non-functional checkbox:

Checkbox Name Emoji
- [ ] ESLint ☑️

Raw Markdown:

| Checkbox | Name | Emoji |
| --- | --- | --- |
| - [ ] | ESLint | ☑️ |
HonkingGoose commented 3 years ago

Maybe move the emojis after checkbox for better alignment?

I like it better when the text is properly left-aligned. Also this way we can also add a Merge Confidence "confidence" badge at the end of the text as well:

viceice commented 3 years ago

Ok, agreed. Was just an idea. 🤗

HonkingGoose commented 3 years ago

Stupid idea incoming, but would it be cleaner to removate the semantic commit prefixes, or even more text?

We can get pretty long lines as is, and those lines will only get longer if we eventually add Merge Confidence badges...

rarkin's mockup:

Remove semantic commit prefixes

Remove semantic commit prefixes + update text

rarkins commented 3 years ago

"Remove semantic commit prefixes" is maybe a possibility. Maybe we can make it templated so that users can pick either.

viceice commented 3 years ago

Sounds good. I like the idea to have the text as short as possible.

HonkingGoose commented 3 years ago

@rarkins Do you want a separate issue for the "remove semantic commit prefixes" idea?

rarkins commented 2 years ago

I've hidden a bunch of comments here as I want to remain focused on the branch state caching first, and dashboard reuse of it second

rarkins commented 2 years ago

Blocked by #11426