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

feat: job summary #13

Conversation

gentlementlegen
Copy link
Member

QA: https://github.com/Meniole/automated-merging/actions/runs/10484217034

This adds a summary on the job which is very useful to see what got merged or not without having to go through the logs.

@gentlementlegen gentlementlegen marked this pull request as ready for review August 21, 2024 05:50
src/handlers/summary.ts Outdated Show resolved Hide resolved
src/plugin.ts Outdated Show resolved Hide resolved
@Keyrxng
Copy link
Contributor

Keyrxng commented Aug 21, 2024

This is nice I hadn't considered outputting log summaries before

Copy link
Member

@whilefoo whilefoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great and helps to debug!

src/helpers/github.ts Show resolved Hide resolved
@0x4007
Copy link
Member

0x4007 commented Aug 22, 2024

Still doing minor refinements but perhaps you can get started by adjusting to the following style:

Preview

Merge Report

Status ID
🔵 assistive-pricing#3
⚫️ ubiquibot-kernel#5
⚫️ command-wallet#7
⚫️ command-start-stop#4
⚫️ test-automated-merging#1
⚫️ automated-merging#1
⚫️ command-wallet#1
⚫️ user-activity-watcher#3
⚫️ command-query-user#3
⚫️ ts-template#6

🔵 = merged;
⚫️ = unmerged

Configuration

Watching Repositories

Ignored Repositories

Source Code

## Merge Report

<samp>



| Status | ID |
|--------|-----|
| 🔵      | [assistive-pricing#3](https://github.com/Meniole/assistive-pricing/pull/3) |
| ⚫️     | [ubiquibot-kernel#5](https://github.com/Meniole/ubiquibot-kernel/pull/5) |
| ⚫️     | [command-wallet#7](https://github.com/Meniole/command-wallet/pull/7) |
| ⚫️     | [command-start-stop#4](https://github.com/Meniole/command-start-stop/pull/4) |
| ⚫️     | [test-automated-merging#1](https://github.com/Meniole/test-automated-merging/pull/1) |
| ⚫️     | [automated-merging#1](https://github.com/Meniole/automated-merging/pull/1) |
| ⚫️     | [command-wallet#1](https://github.com/Meniole/command-wallet/pull/1) |
| ⚫️     | [user-activity-watcher#3](https://github.com/Meniole/user-activity-watcher/pull/3) |
| ⚫️     | [command-query-user#3](https://github.com/Meniole/command-query-user/pull/3) |
| ⚫️     | [ts-template#6](https://github.com/Meniole/ts-template/pull/6) |

🔵 = merged;
⚫️ = unmerged

</samp>



## Configuration
### Watching Repositories
### Ignored Repositories
- [conversation-rewards](https://github.com/ubiquibot/conversation-rewards)

@gentlementlegen
Copy link
Member Author

@gentlementlegen gentlementlegen requested a review from 0x4007 August 22, 2024 06:41
@0x4007
Copy link
Member

0x4007 commented Aug 22, 2024

That's too bad i can't seem to see anything from mobile

image

@gentlementlegen
Copy link
Member Author

image

@0x4007
Copy link
Member

0x4007 commented Aug 22, 2024

I assume it's fine so we can merge.

@gentlementlegen gentlementlegen merged commit e0f4b64 into ubiquity-os-marketplace:development Aug 22, 2024
2 checks passed
@gentlementlegen gentlementlegen deleted the feat/job-summary branch August 22, 2024 06:44
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.

4 participants