Skip to content
This repository has been archived by the owner on Jun 12, 2024. It is now read-only.

Change approvals to include reviewer tier #130

Merged
merged 19 commits into from
Jun 28, 2023
Merged

Conversation

prince-chrismc
Copy link
Owner

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Jun 26, 2023

Codecov Report

Merging #130 (a5f030b) into main (a437a2f) will increase coverage by 0.85%.
The diff coverage is 72.00%.

@@            Coverage Diff             @@
##             main     #130      +/-   ##
==========================================
+ Coverage   55.57%   56.42%   +0.85%     
==========================================
  Files          18       18              
  Lines         619      638      +19     
==========================================
+ Hits          344      360      +16     
- Misses        239      241       +2     
- Partials       36       37       +1     
Impacted Files Coverage Δ
pending_review/known_reviewers.go 64.70% <ø> (ø)
internal/format/reviews.go 84.46% <57.14%> (-4.55%) ⬇️
pending_review/review.go 70.37% <90.90%> (+6.85%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@prince-chrismc

This comment was marked as outdated.

@prince-chrismc

This comment was marked as outdated.

@prince-chrismc
Copy link
Owner Author

Turns out this is not possible https://stackoverflow.com/questions/11509830/how-to-add-color-to-githubs-readme-md-file wihtout any workarounds

@prince-chrismc prince-chrismc changed the title Change approvals to include official status Change approvals to include reviewer tier Jun 26, 2023
@prince-chrismc prince-chrismc marked this pull request as ready for review June 26, 2023 22:41
@prince-chrismc
Copy link
Owner Author

✨ Summary of Pull Requests Pending Review!

☑️ Selection Criteria:

  • There has been at least one approval on the head commit
  • The last commit occurred after any reviews
  • Must not have a label indicating stopped or auto merge

Legend

Icon Description
🆕 Adding a recipe which does not yet exist
📝 Modification to an existing recipe
📗 Documentation change [1]
⚙️ GitHub configuration/workflow changes [1]
⏱️ or ⚠️ The commit status does not indicate success [2]
🔔 The last review was more than 12 days ago
👀 It's been more than 3 days since the last commit and there are no reviews
  • [1]: closely matches the label
  • [2]: depending whether the PR is under way or ready to merge

🤓 Please Review!

There are 85 pull requests currently under way 🕵️

PR By Opened Recipe Weight Reviews Last 🛑 Blockers 🌟 Approvers
#10776 paulharris May 13 ⏱️ vtk XL 4 Dec 1 🔔
#14509 toge Dec 1 ⏱️ cassandra-cpp-driver XL 0 👀
#14916 System-Arch Dec 23 ⏱️ isl L 3 Dec 24 🔔
#15193 SpaceIm Jan 10 ⏱️ opencv XL 4 May 27 🔔
#15678 sabelka Feb 2 ⏱️ qt M 4 Feb 14 🔔
#15726 Hopobcn Feb 4 ⏱️ folly XL 12 Apr 17 🔔
#16098 toge Feb 17 📝 protobuf M 21 Jun 26 miklelappo, coryan
#16134 RazielXYZ Feb 19 ⏱️ bgfx L 5 Apr 28 🔔
#16222 System-Arch Feb 22 ⏱️ boost 🟦 S 8 May 19 🔔
#16313 hellozee Feb 27 ⏱️ qt M 7 Apr 26 🔔 ericLemanissier, jwillikers
#16320 toge Feb 27 📝 osmanip L 0 👀
#16432 jave27 Mar 7 ⏱️ libwebsockets 🟢 XS 0 👀
#16528 psyinf Mar 11 🆕 vsg L 20 Jun 27 RubenRBS franramirez688
#16587 nicosmd Mar 16 ⏱️ opencv 🟢 XS 0 👀
#16589 cynix Mar 16 ⏱️ openldap 🟢 XS 0 👀
#16592 prince-chrismc Mar 17 ⏱️ docs 🟢 XS 8 Apr 1 🔔
#16679 swebb2066 Mar 24 ⏱️ log4cxx XL 5 May 8 🔔
#16698 Woazim Mar 25 ⏱️ imagl L 4 Apr 11 🔔
#16738 kuroneko Mar 27 ⏱️ cglm L 1 Jun 23
#16872 toge Apr 3 ⏱️ drogon L 0
#17025 SSE4 Apr 14 🆕 vvenc M 3 Apr 14 🔔 uilianries
#17032 datalogics-kam Apr 14 ⏱️ cmake L 0 👀
#17108 SpaceIm Apr 18 ⏱️ aws-c-auth M 2 Jun 8 🔔 jwillikers, mpimenov
#17110 SpaceIm Apr 18 ⏱️ aws-c-event-stream M 2 Jun 8 🔔 jwillikers, mpimenov
#17111 SpaceIm Apr 18 ⏱️ aws-c-mqtt M 2 Jun 8 🔔 jwillikers, mpimenov
#17242 hedtke Apr 26 ⏱️ scip M 4 Jun 6 🔔
#17272 0xFireWolf Apr 27 📝 elfutils L 5 Jun 27 ericLemanissier, jwillikers
#17274 SpaceIm Apr 27 📝 sentry-native 🟦 S 4 Jun 27 MartinDelille
#17338 SpaceIm May 1 ⏱️ coin-lemon L 5 May 20 🔔 jcar87 jwillikers, prince-chrismc, SSE4
#17367 Corallo May 2 ⏱️ opencv 🟢 XS 0 👀
#17395 SpaceIm May 4 ⏱️ h5pp L 0 👀
#17408 toge May 5 🆕 sonic-cpp M 8 Jun 27 RubenRBS
#17414 valgur May 5 ⏱️ eastl L 7 Jun 16 memsharded
#17423 Linux13524 May 6 ⏱️ onnxruntime 🟢 XS 0 👀
#17427 samuel-emrys May 6 ⏱️ tk L 0 👀
#17436 SpaceIm May 7 ⏱️ openmvg 🟢 XS 0 👀
#17442 mtudan May 7 ⏱️ paho-mqtt-c 🟢 XS 0 👀
#17457 jwillikers May 8 ⏱️ libtool 🟦 S 0 👀
#17485 SpaceIm May 9 ⏱️ libxft L 2 Jun 22 jwillikers, SSE4
#17505 Seadex-GmbH May 10 ⏱️ seadex-genesis M 3 Jun 26
#17509 jusito May 10 ⏱️ llvm XL 0 👀
#17528 kammce May 11 ⏱️ arm-gnu-toolchain L 6 Jun 25 jwillikers
#17542 valgur May 12 ⏱️ stdgpu L 0 👀
#17575 valgur May 15 ⏱️ embag M 1 May 25 🔔
#17580 miklelappo May 15 ⏱️ grpc L 0 👀
#17609 swebb2066 May 18 ⏱️ apr L 0 👀
#17636 vbieleny May 20 ⏱️ android-ndk 🟦 S 0 👀
#17655 jwillikers May 22 ⏱️ pango L 0 👀
#17682 uilianries May 24 ⏱️ chaiscript L 0 👀
#17707 kevswims May 25 🆕 foxglove-schemas-protobuf M 4 Jun 26
#17719 rturrado May 26 ⏱️ libqasm M 12 Jun 27 RubenRBS SSE4
#17722 0xFireWolf May 27 ⏱️ jemalloc L 1 May 31 🔔
#17725 gjasny May 27 📝 protobuf 🟦 S 5 Jun 26 Hopobcn, jwillikers
#17728 YaZasnyal May 28 ⏱️ ffmpeg 🟦 S 0 👀
#17761 leha-bot May 30 ⏱️ ccache 🟢 XS 0 👀
#17768 ericLemanissier May 31 ⏱️ mingw-builds 🟦 S 0 👀
#17784 Sa3eedove Jun 1 ⏱️ usockets L 0 👀
#17806 samuel-emrys Jun 3 ⏱️ libdb L 0 👀
#17825 mzdun Jun 5 🆕 mbits-lngs L 1 Jun 27
#17868 toge Jun 8 🆕 maven M 1 Jun 26 jwillikers
#17881 fcorso2016 Jun 9 ⏱️ whisper-cpp L 1 Jun 14 🔔 RubenRBS
#17882 PeteAudinate Jun 9 ⏱️ b2 🟦 S 1 Jun 25 jwillikers
#17894 sourcedelica Jun 10 ⏱️ librdkafka 🟦 S 0 👀
#17899 xahon Jun 11 ⏱️ nativefiledialog 🟢 XS 0 👀
#17911 nbukuli Jun 12 ⏱️ linux-headers-generic 🟢 XS 0 👀
#17916 patmantru Jun 12 ⏱️ opentdf-client 🟦 S 1 Jun 14 🔔
#17925 MartinDelille Jun 13 ⏱️ qt L 2 Jun 25 jwillikers
#17933 valgur Jun 14 📝 gtsam XL 0
#17972 valgur Jun 19 📝 metis XL 0
#17976 pwqbot Jun 20 📝 spdlog M 1 Jun 27 jcar87
#17995 ericLemanissier Jun 21 📝 fontconfig 🟢 XS 1 Jun 24 jwillikers
#18027 keyboardspecialist Jun 22 ⏱️ gdal L 0 👀
#18034 Cogitri Jun 23 📝 opentelemetry-cpp 🟦 S 0
#18047 mayeut Jun 24 📝 libpng 🟦 S 2 Jun 26
#18058 maksim-petukhov Jun 26 📝 m4 🟦 S 1 Jun 26 jwillikers
#18059 maksim-petukhov Jun 26 📝 autoconf 🟢 XS 1 Jun 26 jwillikers
#18069 toge Jun 27 📝 sfml L 4 Jun 27
#18070 LevWi Jun 27 ⏱️ openh264 🟢 XS 0
#18073 holger-detering Jun 27 📝 cute_headers M 1 Jun 27
#18082 RubenRBS Jun 27 ⚙️ .c3i 🟢 XS 0
#18083 toge Jun 27 📝 rapidcsv 🟦 S 1 Jun 27 RubenRBS
#18085 toge Jun 27 📝 r8brain-free-src 🟦 S 1 Jun 27
#18088 maksim-petukhov Jun 27 ⏱️ libtool 🟦 S 0
#18091 valgur Jun 27 📝 certify L 2 Jun 27
#18093 valgur Jun 27 📝 continuable L 1 Jun 27 RubenRBS

✔️ Ready to Merge

Currently 4 pull requests are waiting to be merged 🎉

PR By Opened Recipe Reviews 🌟 Approvers
#16613 SpaceIm Mar 18 ⚠️ librasterlite2 2 samuel-emrys, uilianries
#17691 mpimenov May 24 ⚠️ arrow 2 uilianries, SSE4
#17752 0xFireWolf May 30 📝 wolfssl 7 prince-chrismc, czoido
#17994 valgur Jun 21 ⚠️ sbp 4 RubenRBS, SSE4, jcar87

📋 Statistics

⚠️ These are just rough metrics counting the labels and may not reflect the actual state of pull requests

  • Commit:
  • Pull Requests:
    • Open: 233
    • Draft: 31
    • Average Age: 98 days, 8 hours, and 6.66 minutes
    • Stop Label: 98

Raw JSON data

📊 Open Versus Merged

Legend

Pull requests are depicted as:

  • Open 🟩
  • Closed 🟥
  • Merged 🟪
    • Darker bottom section indicated merged within 7 days of being opened

For reference:

  • 100% is 206 (most in the last year)
  • 60% is 123

ovm

⌛ Time Spent in Review

If you are wondering how long it will take for you pull request to get merged; this graph should give you an idea.

tir

Found this useful? Give it a ⭐ 🙏

@prince-chrismc prince-chrismc merged commit 74fa0aa into main Jun 28, 2023
@prince-chrismc prince-chrismc deleted the prince-chrismc-patch-1 branch June 28, 2023 05:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants