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: provide repository insights #38

Merged
merged 1 commit into from
Oct 2, 2023

Conversation

cecobask
Copy link
Contributor

@cecobask cecobask commented Sep 15, 2023

Description

This PR adds the feature to provide repository insights.

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation Update
  • 🎨 Style
  • πŸ§‘β€πŸ’» Code Refactor
  • πŸ”₯ Performance Improvements
  • βœ… Test
  • πŸ€– Build
  • πŸ” CI
  • πŸ“¦ Chore (Release)
  • ⏩ Revert

Related Tickets & Documents

Closes #10

Mobile & Desktop Screenshots/Recordings

table output
$ pizza insights repositories https://github.com/kubernetes/kubernetes https://github.com/cli/cli -o table

Repository URL         https://github.com/cli/cli 
Repository ID          212613049                  
All pull requests      63                         
Accepted pull requests 33                         
Spam pull requests     0                          
Contributors           47                         
―――
Repository URL         https://github.com/kubernetes/kubernetes 
Repository ID          20580498                                 
All pull requests      990                                      
Accepted pull requests 435                                      
Spam pull requests     0                                        
Contributors           334
json output
$ pizza insights repositories https://github.com/kubernetes/kubernetes https://github.com/cli/cli -o json

[
  {
    "repo_url": "https://github.com/cli/cli",
    "repo_id": 212613049,
    "all_pull_requests": 63,
    "accepted_pull_requests": 33,
    "spam_pull_requests": 0,
    "contributors": [
      "dmgardiner25",
      "peterramaldes",
      "andyfeller",
      "dependabot",
      "siketyan",
      "heaths",
      "john-h-k",
      "rajhawaldar",
      "sarrocks1",
      "arunsathiya",
      "benoit-pierre",
      "samcoe",
      "williammartin",
      "kbailey4444",
      "utouto97",
      "junaga",
      "joshkraft",
      "paulober",
      "Bunyuh",
      "bronislav",
      "joshmgross",
      "Ericsin13",
      "elldritch",
      "mislav",
      "jladbrook",
      "notomo",
      "mik3y",
      "satotake",
      "YorikSar",
      "shanduur",
      "JunNishimura",
      "ncalteen",
      "kemingy",
      "HARI-124",
      "harveysanders",
      "armandgrillet",
      "vilmibm",
      "rwe",
      "mntlty",
      "wilso199",
      "Wakoylabor",
      "Yhamskie30",
      "PappiiTeamo",
      "IsabelSchoepd",
      "ffalor",
      "GO-Institute",
      "kousikmitra"
    ]
  },
  {
    "repo_url": "https://github.com/kubernetes/kubernetes",
    "repo_id": 20580498,
    "all_pull_requests": 990,
    "accepted_pull_requests": 435,
    "spam_pull_requests": 0,
    "contributors": [
      "seans3",
      "liggitt",
      "Jefftree",
      "ardaguclu",
      "princepereira",
      "cvvz",
      "atiratree",
      "sunnylovestiramisu",
      "aojea",
      "hoskeri",
      "JoelSpeed",
      "mimowo",
      "nikhita",
      "afritzler",
      "marseel",
      "daschott",
      "howardjohn",
      "rphillips",
      "xh4n3",
      "alculquicondor",
      "aleksandra-malinowska",
      "jiahuif",
      "jupblb",
      "simonjpartridge",
      "harshanarayana",
      "wojtek-t",
      "ehashman",
      "robscott",
      "ahg-g",
      "serathius",
      "bobbypage",
      "saschagrunert",
      "pegasas",
      "skitt",
      "aramase",
      "shijinye",
      "chansuke",
      "Karthik-K-N",
      "neolit123",
      "kannon92",
      "dims",
      "danwinship",
      "gjkim42",
      "adrianreber",
      "caoyingjunz",
      "carlory",
      "soltysh",
      "p0lyn0mial",
      "mochizuki875",
      "elezar",
      "Gekko0114",
      "ritazh",
      "cyclinder",
      "HirazawaUi",
      "harche",
      "togettoyou",
      "aroradaman",
      "tkashem",
      "pohly",
      "ruquanzhao",
      "swatisehgal",
      "sushanth0910",
      "AhmedGrati",
      "Riaankl",
      "rata",
      "munnerz",
      "eiffel-fl",
      "olderTaoist",
      "TommyStarK",
      "uablrek",
      "adilGhaffarDev",
      "li7819559",
      "fatsheep9146",
      "ibilalkayy",
      "weilaaa",
      "bzsuni",
      "thockin",
      "jkh52",
      "andrewsykim",
      "yt2985",
      "machine424",
      "akhilerm",
      "apelisse",
      "kwakubiney",
      "bertinatto",
      "likakuli",
      "lvedder",
      "lzhecheng",
      "xiaoanyunfei",
      "iholder101",
      "cfanbo",
      "sbdtu5498",
      "PI-Victor",
      "wzshiming",
      "tnqn",
      "SataQiu",
      "palnabarun",
      "ipochi",
      "pacoxu",
      "wackxu",
      "RomanBednar",
      "linxiulei",
      "akankshapanse",
      "liyuerich",
      "Affan-7",
      "bells17",
      "dastonzerg",
      "tukwila",
      "b8kings0ga",
      "fusida",
      "rokkiter",
      "dpasiukevich",
      "dhruvmehtaaa",
      "sanposhiho",
      "Dashhh",
      "alexzielenski",
      "rohitssingh",
      "AryanSharma9917",
      "ls-2018",
      "my-git9",
      "f4nd0y",
      "dzjiang91",
      "aheng-ch",
      "MikeSpreitzer",
      "wedsonaf",
      "AliDatadog",
      "mowangdk",
      "ffromani",
      "newtondev",
      "shawnhanx",
      "alvaroaleman",
      "tangwz",
      "Fish-pro",
      "MadhavJivrajani",
      "cezarygerard",
      "tenzen-y",
      "rexagod",
      "wangwei10061",
      "ruiwen-zhao",
      "Richabanker",
      "mengjiao-liu",
      "tnyeanderson",
      "yuanchen8911",
      "charles-chenzz",
      "sxllwx",
      "Rei1010",
      "wlq1212",
      "chendave",
      "Lukasz-AWS",
      "sairameshv",
      "champtar",
      "cuishuang",
      "YuikoTakada",
      "matte21",
      "lixd",
      "akash-kumar-saw",
      "Mskxn",
      "keisukesakasai",
      "pperiyasamy",
      "sozercan",
      "enj",
      "Tal-or",
      "ArkaSaha30",
      "upodroid",
      "rothgar",
      "poneding",
      "rpluem-vf",
      "mjudeikis",
      "srivastav-abhishek",
      "benluddy",
      "rayowang",
      "BenTheElder",
      "gauravkghildiyal",
      "mukesh-mahato",
      "MaGaroo",
      "davidmccormick",
      "sonasingh46",
      "sstoner",
      "ljosyula",
      "AxeZhan",
      "xrstf",
      "jcchavezs",
      "haircommander",
      "Ithrael",
      "fsmunoz",
      "SaumyaBhushan",
      "jlsong01",
      "mkowalski",
      "liangyuanpeng",
      "kerthcet",
      "rxy0210",
      "vinaykul",
      "seeewhy",
      "mansi1597",
      "nayihz",
      "iiiceoo",
      "ADustyOldMuffin",
      "qiutongs",
      "fengxsong",
      "sttts",
      "justinsb",
      "tzneal",
      "logicalhan",
      "kaovilai",
      "gccio",
      "testwill",
      "mauriciopoppe",
      "ahmedtd",
      "kl52752",
      "humblec",
      "mkumatag",
      "horacexd",
      "mlowery",
      "seantywork",
      "muff1nman",
      "jsafrane",
      "haoruan",
      "lojies",
      "devincd",
      "dgrisonnet",
      "mxpv",
      "ialidzhikov",
      "Xunzhuo",
      "qingwave",
      "MarSik",
      "Rajalakshmi-Girish",
      "bart0sh",
      "azylinski",
      "jayunit100",
      "Ritikaa96",
      "wking",
      "fabi200123",
      "sohankunkerkar",
      "Sharpz7",
      "cpanato",
      "deepakkinni",
      "MartinForReal",
      "yanghesong",
      "DamianSawicki",
      "adrianchiris",
      "yanhongchang",
      "claudiubelu",
      "tommas1988",
      "pbxqdown",
      "nilekhc",
      "Huang-Wei",
      "tosi3k",
      "mborsz",
      "Abirdcfly",
      "smarterclayton",
      "gautierdelorme",
      "brianpursley",
      "googs1025",
      "claassen",
      "ty-dc",
      "jprzychodzen",
      "stephenfin",
      "lance5890",
      "lanycrost",
      "philchia",
      "xyz-li",
      "Miciah",
      "cr7258",
      "Penguin-zlh",
      "dashpole",
      "sreeram-venkitesh",
      "jperkin",
      "tjungblu",
      "YaoC",
      "letmerecall",
      "cndoit18",
      "lowang-bh",
      "249043822",
      "stevekuznetsov",
      "fmuyassarov",
      "ushitora-anqou",
      "andrewshaoyu",
      "icefed",
      "jpbetz",
      "kmala",
      "cartermckinnon",
      "LingyanYin",
      "xvzf",
      "HeGaoYuan",
      "pawbana",
      "Gar-nation",
      "Gauravpadam",
      "ionutbalutoiu",
      "Napsty",
      "aofekiko",
      "lauchokyip",
      "anandanthony",
      "weizhoublue",
      "dgl",
      "Cool-Coder174",
      "sunnoy",
      "marquiz",
      "ASverdlov",
      "hwdef",
      "rjsadow",
      "DrAuYueng",
      "ning0515",
      "nazarewk",
      "pmalek",
      "arienshibani",
      "Songjoy",
      "apricote",
      "rayandas",
      "dairlair",
      "vbatts",
      "xuzhenglun",
      "elmiko",
      "Jamstah",
      "zhuqi-lucas",
      "juniorsaldanha",
      "j4ckstraw",
      "GraceAtwood",
      "kaminn",
      "dejanzele",
      "SoorajBharadwaj07",
      "lengrongfu",
      "24sama",
      "rakshitgondwal",
      "yanjing1104",
      "LinuxSuRen",
      "vinayakankugoyal",
      "aledbf",
      "saurabh30115",
      "praveenkprabhakaran",
      "abhilasha2418",
      "champly",
      "nine03",
      "deads2k",
      "donggangcj"
    ]
  }
]
yaml output
$ pizza insights repositories https://github.com/kubernetes/kubernetes https://github.com/cli/cli -o yaml

- repo_url: https://github.com/cli/cli
  repo_id: 212613049
  all_pull_requests: 63
  accepted_pull_requests: 33
  spam_pull_requests: 0
  contributors:
    - dmgardiner25
    - peterramaldes
    - andyfeller
    - dependabot
    - siketyan
    - heaths
    - john-h-k
    - rajhawaldar
    - sarrocks1
    - arunsathiya
    - benoit-pierre
    - samcoe
    - williammartin
    - kbailey4444
    - utouto97
    - junaga
    - joshkraft
    - paulober
    - Bunyuh
    - bronislav
    - joshmgross
    - Ericsin13
    - elldritch
    - mislav
    - jladbrook
    - notomo
    - mik3y
    - satotake
    - YorikSar
    - shanduur
    - JunNishimura
    - ncalteen
    - kemingy
    - HARI-124
    - harveysanders
    - armandgrillet
    - vilmibm
    - rwe
    - mntlty
    - wilso199
    - Wakoylabor
    - Yhamskie30
    - PappiiTeamo
    - IsabelSchoepd
    - ffalor
    - GO-Institute
    - kousikmitra
- repo_url: https://github.com/kubernetes/kubernetes
  repo_id: 20580498
  all_pull_requests: 990
  accepted_pull_requests: 435
  spam_pull_requests: 0
  contributors:
    - seans3
    - liggitt
    - Jefftree
    - ardaguclu
    - princepereira
    - cvvz
    - atiratree
    - sunnylovestiramisu
    - aojea
    - hoskeri
    - JoelSpeed
    - mimowo
    - nikhita
    - afritzler
    - marseel
    - daschott
    - howardjohn
    - rphillips
    - xh4n3
    - alculquicondor
    - aleksandra-malinowska
    - jiahuif
    - jupblb
    - simonjpartridge
    - harshanarayana
    - wojtek-t
    - ehashman
    - robscott
    - ahg-g
    - serathius
    - bobbypage
    - saschagrunert
    - pegasas
    - skitt
    - aramase
    - shijinye
    - chansuke
    - Karthik-K-N
    - neolit123
    - kannon92
    - dims
    - danwinship
    - gjkim42
    - adrianreber
    - caoyingjunz
    - carlory
    - soltysh
    - p0lyn0mial
    - mochizuki875
    - elezar
    - Gekko0114
    - ritazh
    - cyclinder
    - HirazawaUi
    - harche
    - togettoyou
    - aroradaman
    - tkashem
    - pohly
    - ruquanzhao
    - swatisehgal
    - sushanth0910
    - AhmedGrati
    - Riaankl
    - rata
    - munnerz
    - eiffel-fl
    - olderTaoist
    - TommyStarK
    - uablrek
    - adilGhaffarDev
    - li7819559
    - fatsheep9146
    - ibilalkayy
    - weilaaa
    - bzsuni
    - thockin
    - jkh52
    - andrewsykim
    - yt2985
    - machine424
    - akhilerm
    - apelisse
    - kwakubiney
    - bertinatto
    - likakuli
    - lvedder
    - lzhecheng
    - xiaoanyunfei
    - iholder101
    - cfanbo
    - sbdtu5498
    - PI-Victor
    - wzshiming
    - tnqn
    - SataQiu
    - palnabarun
    - ipochi
    - pacoxu
    - wackxu
    - RomanBednar
    - linxiulei
    - akankshapanse
    - liyuerich
    - Affan-7
    - bells17
    - dastonzerg
    - tukwila
    - b8kings0ga
    - fusida
    - rokkiter
    - dpasiukevich
    - dhruvmehtaaa
    - sanposhiho
    - Dashhh
    - alexzielenski
    - rohitssingh
    - AryanSharma9917
    - ls-2018
    - my-git9
    - f4nd0y
    - dzjiang91
    - aheng-ch
    - MikeSpreitzer
    - wedsonaf
    - AliDatadog
    - mowangdk
    - ffromani
    - newtondev
    - shawnhanx
    - alvaroaleman
    - tangwz
    - Fish-pro
    - MadhavJivrajani
    - cezarygerard
    - tenzen-y
    - rexagod
    - wangwei10061
    - ruiwen-zhao
    - Richabanker
    - mengjiao-liu
    - tnyeanderson
    - yuanchen8911
    - charles-chenzz
    - sxllwx
    - Rei1010
    - wlq1212
    - chendave
    - Lukasz-AWS
    - sairameshv
    - champtar
    - cuishuang
    - YuikoTakada
    - matte21
    - lixd
    - akash-kumar-saw
    - Mskxn
    - keisukesakasai
    - pperiyasamy
    - sozercan
    - enj
    - Tal-or
    - ArkaSaha30
    - upodroid
    - rothgar
    - poneding
    - rpluem-vf
    - mjudeikis
    - srivastav-abhishek
    - benluddy
    - rayowang
    - BenTheElder
    - gauravkghildiyal
    - mukesh-mahato
    - MaGaroo
    - davidmccormick
    - sonasingh46
    - sstoner
    - ljosyula
    - AxeZhan
    - xrstf
    - jcchavezs
    - haircommander
    - Ithrael
    - fsmunoz
    - SaumyaBhushan
    - jlsong01
    - mkowalski
    - liangyuanpeng
    - kerthcet
    - rxy0210
    - vinaykul
    - seeewhy
    - mansi1597
    - nayihz
    - iiiceoo
    - ADustyOldMuffin
    - qiutongs
    - fengxsong
    - sttts
    - justinsb
    - tzneal
    - logicalhan
    - kaovilai
    - gccio
    - testwill
    - mauriciopoppe
    - ahmedtd
    - kl52752
    - humblec
    - mkumatag
    - horacexd
    - mlowery
    - seantywork
    - muff1nman
    - jsafrane
    - haoruan
    - lojies
    - devincd
    - dgrisonnet
    - mxpv
    - ialidzhikov
    - Xunzhuo
    - qingwave
    - MarSik
    - Rajalakshmi-Girish
    - bart0sh
    - azylinski
    - jayunit100
    - Ritikaa96
    - wking
    - fabi200123
    - sohankunkerkar
    - Sharpz7
    - cpanato
    - deepakkinni
    - MartinForReal
    - yanghesong
    - DamianSawicki
    - adrianchiris
    - yanhongchang
    - claudiubelu
    - tommas1988
    - pbxqdown
    - nilekhc
    - Huang-Wei
    - tosi3k
    - mborsz
    - Abirdcfly
    - smarterclayton
    - gautierdelorme
    - brianpursley
    - googs1025
    - claassen
    - ty-dc
    - jprzychodzen
    - stephenfin
    - lance5890
    - lanycrost
    - philchia
    - xyz-li
    - Miciah
    - cr7258
    - Penguin-zlh
    - dashpole
    - sreeram-venkitesh
    - jperkin
    - tjungblu
    - YaoC
    - letmerecall
    - cndoit18
    - lowang-bh
    - "249043822"
    - stevekuznetsov
    - fmuyassarov
    - ushitora-anqou
    - andrewshaoyu
    - icefed
    - jpbetz
    - kmala
    - cartermckinnon
    - LingyanYin
    - xvzf
    - HeGaoYuan
    - pawbana
    - Gar-nation
    - Gauravpadam
    - ionutbalutoiu
    - Napsty
    - aofekiko
    - lauchokyip
    - anandanthony
    - weizhoublue
    - dgl
    - Cool-Coder174
    - sunnoy
    - marquiz
    - ASverdlov
    - hwdef
    - rjsadow
    - DrAuYueng
    - ning0515
    - nazarewk
    - pmalek
    - arienshibani
    - Songjoy
    - apricote
    - rayandas
    - dairlair
    - vbatts
    - xuzhenglun
    - elmiko
    - Jamstah
    - zhuqi-lucas
    - juniorsaldanha
    - j4ckstraw
    - GraceAtwood
    - kaminn
    - dejanzele
    - SoorajBharadwaj07
    - lengrongfu
    - 24sama
    - rakshitgondwal
    - yanjing1104
    - LinuxSuRen
    - vinayakankugoyal
    - aledbf
    - saurabh30115
    - praveenkprabhakaran
    - abhilasha2418
    - champly
    - nine03
    - deads2k
    - donggangcj

Added tests?

  • πŸ‘ yes
  • πŸ™… no, because they aren't needed
  • πŸ™‹ no, because I need help

Added to documentation?

  • πŸ“œ README.md
  • πŸ““ docs.opensauced.pizza
  • πŸ• dev.to/opensauced
  • πŸ“• storybook
  • πŸ™… no documentation needed

@cecobask cecobask changed the title feat: provide repository insights Draft: feat: provide repository insights Sep 15, 2023
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

PR Compliance Checks Passed!

@cecobask cecobask changed the title Draft: feat: provide repository insights feat: provide repository insights Sep 15, 2023
@cecobask cecobask marked this pull request as draft September 15, 2023 14:51
@cecobask cecobask force-pushed the repository-insights branch 5 times, most recently from 5bf5854 to 0f0f1dc Compare September 27, 2023 16:05
@cecobask cecobask marked this pull request as ready for review September 27, 2023 16:08
@cecobask cecobask force-pushed the repository-insights branch 2 times, most recently from 39a9646 to 565934f Compare September 27, 2023 16:36
@cecobask
Copy link
Contributor Author

@jpmcb @bdougie This pull request is ready for review.

Copy link
Member

@jpmcb jpmcb left a comment

Choose a reason for hiding this comment

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

Awesome work. Overall this looks great πŸ‘πŸΌ will tag others for more feedback

README.md Show resolved Hide resolved
cmd/insights/contributors.go Outdated Show resolved Hide resolved
cmd/insights/contributors.go Outdated Show resolved Hide resolved
cmd/insights/repositories.go Outdated Show resolved Hide resolved
cmd/insights/repositories.go Outdated Show resolved Hide resolved
cmd/insights/contributors.go Show resolved Hide resolved
@jpmcb
Copy link
Member

jpmcb commented Sep 27, 2023

CC @k1nho you might find this interesting/useful since it's some code you recently touched

@jpmcb jpmcb requested a review from a team September 27, 2023 19:49
Copy link
Member

@nickytonline nickytonline left a comment

Choose a reason for hiding this comment

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

I still need to get the pizza-cli running locally, so just a surface comment for the moment.

cmd/insights/contributors.go Show resolved Hide resolved
@cecobask
Copy link
Contributor Author

cecobask commented Sep 28, 2023

Awesome work. Overall this looks great πŸ‘πŸΌ will tag others for more feedback

@jpmcb Thank you! I've updated the code based on your suggestions πŸ‘

Copy link
Member

@jpmcb jpmcb left a comment

Choose a reason for hiding this comment

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

@jpmcb jpmcb requested a review from a team October 2, 2023 20:37
Copy link
Member

@nickytonline nickytonline left a comment

Choose a reason for hiding this comment

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

🚒

@jpmcb jpmcb merged commit dc148d6 into open-sauced:beta Oct 2, 2023
6 checks passed
open-sauced bot pushed a commit that referenced this pull request Oct 2, 2023
## [1.0.0-beta.7](v1.0.0-beta.6...v1.0.0-beta.7) (2023-10-02)

### πŸ• Features

* provide repository insights ([#38](#38)) ([dc148d6](dc148d6))
@cecobask cecobask deleted the repository-insights branch October 2, 2023 23:52
open-sauced bot pushed a commit that referenced this pull request Oct 11, 2023
## 1.0.0 (2023-10-11)

### πŸ€– Build System

* sematic bin release, npm ([7b4607e](7b4607e))

### πŸ” Continuous Integration

* Update @open-sauced/release@2.2.1 and compliance.yaml ([#33](#33)) ([146b6b7](146b6b7))

### πŸ› Bug Fixes

* Uses correct generated token when checking out cli repo in release ([#44](#44)) ([1e0c9f1](1e0c9f1))

### πŸ• Features

* Add install instructions and script for pizza CLI ([#26](#26)) ([421a429](421a429))
* Add posthog telemetry integration ([#37](#37)) ([9829f49](9829f49))
* cli auth ([#21](#21)) ([34728fb](34728fb))
* GitHub action to build and upload Go artifacts after release created ([#22](#22)) ([ad187a9](ad187a9))
* Http Client for accessing OpenSauced API client ([#23](#23)) ([ec2b357](ec2b357))
* Leverage the GITHUB_APP_TOKEN for releases ([#32](#32)) ([e0a25e0](e0a25e0))
* npm i -g pizza ([73291d1](73291d1))
* Pizza show ([#24](#24)) ([72f21ce](72f21ce))
* provide repository contributors insights ([#30](#30)) ([d16091f](d16091f))
* provide repository insights ([#38](#38)) ([dc148d6](dc148d6))
* repo-query support ([199cfd7](199cfd7))
* update bin name release.yaml ([6b21cb8](6b21cb8))
* Version command for CLI based on release builds ([#36](#36)) ([9f3eedc](9f3eedc))
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.

Feature: pizza insights repo for individual repo insights
3 participants