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

Send summaries of PR and merges to main chat #7060

Merged
merged 3 commits into from
Aug 11, 2024
Merged

Conversation

EvilOlaf
Copy link
Member

@EvilOlaf EvilOlaf commented Aug 11, 2024

Description

While having #armbian-commits channels they feel bulky and barely anyone uses/recognizes them. Too much information is sent and sometimes too fast.

This boils it down to a single line that is sent when somebody opens a pull request or a commit is merged, containing link to the sender, the repository and the commit. Also the commit head message is shown.

Notes:

This PR needs preparation at repository admin/owner level since some secrets need to be set like shown here:
https://github.com/EvilOlaf/gha_discord/blob/main/README.md

The same is necessary for any other repository that should be announced.

How Has This Been Tested?

Checklist:

  • I have performed a self-review of my own code
  • My changes generate no new warnings

Sample output:
Selection_103

@github-actions github-actions bot added size/small PR with less then 50 lines GitHub Actions GitHub Actions code labels Aug 11, 2024
@EvilOlaf
Copy link
Member Author

Secrets are missing, therefore the workflow fails.

@EvilOlaf EvilOlaf marked this pull request as ready for review August 11, 2024 07:07
@EvilOlaf
Copy link
Member Author

please set necessary secrets before merging.

@igorpecovnik igorpecovnik added Ready to merge Reviewed, tested and ready for merge 08 Milestone: Third quarter release labels Aug 11, 2024
@EvilOlaf EvilOlaf merged commit 77e6c9c into armbian:main Aug 11, 2024
Dangku pushed a commit to BPI-SINOVOIP/armbian-build that referenced this pull request Jan 16, 2025
* Create pr-announce.yml

* Rename pr-announce.yml to pr-announce.yml

* Create merge-announce.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
08 Milestone: Third quarter release GitHub Actions GitHub Actions code Ready to merge Reviewed, tested and ready for merge size/small PR with less then 50 lines
Development

Successfully merging this pull request may close these issues.

2 participants