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: Add sort capabilities to user contributions #65

Merged
merged 1 commit into from
Nov 21, 2023

Conversation

k1nho
Copy link
Contributor

@k1nho k1nho commented Nov 2, 2023

Description

This PR adds the sort flag to be able to sort the output of insights user-contributions command. Options to sort include (total, commits, prs).

Example (prs)

open-sauced/app
User                Total      Commits    PRs Created     
brandonroberts      328        299        29              
OgDev-01            175        146        29              
nickytonline        183        162        21              
5hraddha            47         40         7               
RitaDee             46         39         7               
itskish0re          14         9          5               
Satoshi-Sh          6          3          3               
shelar1423          6          3          3               
aryanas159          26         24         2               
brainconnect93      9          7          2               
Deadreyo            8          6          2               
Gourav2609          15         13         2               
289Volts            14         13         1               
ATREAY              2          1          1               
babblebey           13         12         1               
CBID2               3          2          1               
diivi               3          2          1               
foxyblocks          8          7          1               
FrancoLab           4          3          1               
Hiperultimate       5          4          1               
Lucif3r-in          3          2          1               
Lulalaby            3          2          1               
ryuk-jayant         3          2          1               
shamimbinnur        6          5          1               
smrkhan21           2          1          1               
SoundaryaKoutharapu 4          3          1               
tarunclub           2          1          1               

Example(commits)

open-sauced/app
User                Total      Commits    PRs Created     
brandonroberts      328        299        29              
nickytonline        183        162        21              
OgDev-01            175        146        29              
5hraddha            47         40         7               
RitaDee             46         39         7               
aryanas159          26         24         2               
289Volts            14         13         1               
Gourav2609          15         13         2               
babblebey           13         12         1               
itskish0re          14         9          5               
brainconnect93      9          7          2               
foxyblocks          8          7          1               
Deadreyo            8          6          2               
shamimbinnur        6          5          1               
Hiperultimate       5          4          1               
FrancoLab           4          3          1               
Satoshi-Sh          6          3          3               
shelar1423          6          3          3               
SoundaryaKoutharapu 4          3          1               
CBID2               3          2          1               
diivi               3          2          1               
Lucif3r-in          3          2          1               
Lulalaby            3          2          1               
ryuk-jayant         3          2          1               
ATREAY              2          1          1               
smrkhan21           2          1          1               
tarunclub           2          1          1               

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

#64 sort output based on column

Mobile & Desktop Screenshots/Recordings

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

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

@k1nho k1nho marked this pull request as ready for review November 6, 2023 16:30
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.

❯ ./build/pizza insights user-contributions https://github.com/open-sauced/app -s total
https://github.com/open-sauced/app
User                Total      Commits    PRs Created
brandonroberts      338        306        32
nickytonline        223        202        21
OgDev-01            165        137        28
5hraddha            58         49         9
RitaDee             52         44         8
aryanas159          26         24         2
itskish0re          22         14         8
brainconnect93      17         15         2
289Volts            14         13         1
Gourav2609          14         13         1
babblebey           13         12         1
Deadreyo            8          6          2
Satoshi-Sh          8          5          3
Hiperultimate       6          5          1
shamimbinnur        6          5          1
shelar1423          6          3          3
FrancoLab           4          3          1
SoundaryaKoutharapu 4          3          1
CBID2               3          2          1
diivi               3          2          1
Lucif3r-in          3          2          1
Lulalaby            3          2          1
ryuk-jayant         3          2          1
ATREAY              2          1          1
smrkhan21           2          1          1
tarunclub           2          1          1

Looks solid 👍🏼 !!!!!!

@jpmcb jpmcb requested a review from a team November 6, 2023 21:30
Copy link
Contributor

@brandonroberts brandonroberts left a comment

Choose a reason for hiding this comment

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

🍕

Copy link
Member

@bdougie bdougie left a comment

Choose a reason for hiding this comment

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

Thanks @k1nho

@jpmcb jpmcb merged commit fa9808a into open-sauced:beta Nov 21, 2023
6 checks passed
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