Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: starred repos implementation #399

Merged
merged 1 commit into from
Dec 7, 2023

Conversation

Andreea-Lupu
Copy link
Contributor

What type of PR is this?

Which issue does this PR fix:

What does this PR do / Why do we need it:

If an issue # is not available please add repro steps and logs from IPAMD/CNI showing the issue:

Testing done on this change:

Automation added to e2e:

Will this break upgrades or downgrades. Has updating a running cluster been tested?:

Does this change require updates to the CNI daemonset config files to work?:

Does this PR introduce any user-facing change?:


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@Andreea-Lupu Andreea-Lupu force-pushed the star_repo branch 2 times, most recently from 667a82d to 9230a50 Compare December 4, 2023 23:30
Copy link
Collaborator

@raulkele raulkele left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link

codecov bot commented Dec 7, 2023

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (e97e04e) 86.93% compared to head (85aaf55) 87.26%.

Files Patch % Lines
src/components/Shared/RepoCard.jsx 89.47% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #399      +/-   ##
==========================================
+ Coverage   86.93%   87.26%   +0.32%     
==========================================
  Files          53       53              
  Lines        1539     1594      +55     
  Branches      406      436      +30     
==========================================
+ Hits         1338     1391      +53     
- Misses        188      190       +2     
  Partials       13       13              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Andreea-Lupu <andreealupu1470@yahoo.com>
@andaaron andaaron merged commit d9370fb into project-zot:main Dec 7, 2023
8 checks passed
@andaaron andaaron linked an issue Dec 8, 2023 that may be closed by this pull request
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Repo cards: show user specific metadata
3 participants