Skip to content
This repository has been archived by the owner on Oct 21, 2022. It is now read-only.

Update 'paradox--package-repo-list' even if the repos hasn't star #179

Closed
wants to merge 2 commits into from
Closed

Update 'paradox--package-repo-list' even if the repos hasn't star #179

wants to merge 2 commits into from

Conversation

PierreTechoueyres
Copy link

Decorrelate update of 'paradox--package-repo-list' from counting stars.

* helpers/paradox-counter.el: decorrelate update of
'paradox--package-repo-list' from counting stars.
Comment on lines 106 to 109
(let ((count (paradox-fetch-star-count .repo)))
(when (numberp count)
(puthash name count paradox--star-count)))
(puthash name .repo paradox--package-repo-list)))
Copy link
Owner

Choose a reason for hiding this comment

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

Looks like the indentation is a bit off here because you're using tabs. Could you switch to spaces?

Copy link
Author

Choose a reason for hiding this comment

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

Hope the last commit fix this.

@Malabarba
Copy link
Owner

Hey there, thanks for the PR.
I'm curious though, are you using this file?
This file is meant as a way for me to populate the stars data cache that paradox uses (which, admittedly, I haven't done in a while)

* helpers/paradox-counter.el: untabify last change.
@PierreTechoueyres
Copy link
Author

May be I'm wrong, but it seem to me this file is used to check the repo is from github before allowing you to retrieve commit list.

@PierreTechoueyres
Copy link
Author

PierreTechoueyres commented May 12, 2020

I think this will fix #176 and #180

@PierreTechoueyres
Copy link
Author

ping ?

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