GitHub doesn't show you many statistics related to a repository, aside from a repository's star count. When evaluating a repository on GitHub, I of...
For further actions, you may consider blocking this person and/or reporting abuse
Don’t forget to read the code! 💯
There’s no better way to evaluate the quality of software than to inspect the actual software itself. With proprietary stuff this isn’t an option, but the beauty of open source is all the code is right there waiting for you to read it. 🙌
People often skip this (paramount) step because reading code can be difficult and time consuming. Here are a few questions you can ask yourself about a project, to get you started:
The benefits of this process are immense:
As a wise Jedi once advised his Padewan:
Great point! I never even thought of actually looking at the code, lol.
Especially since there was a bit about how recruiters look at stars. The vast majority of devs that have hobby githubs aren't going to be doing it for the stars!
I'd want to know if I'm hiring someone who writes good code when no one's watching rather than popular code with a lot of eyes on it.
Most programmers don't have any (or any non-trivial) code in public, so it is a big bonus to see a programmer with a popular repository.
If nothing else, it's an indicator that the programmer...
Stars aren't a good indicator of quality, but they do demonstrate a programmer's ability to manage a project to some degree. They're another piece of evidence in the big pool of things that could be considered.
I think going as far as filtering for developers based on their Github repo star counts is not a good idea.
I was on an interview once where the CEO anc CTO tried to figure out if I write good code or not. I mentioned a Github repo of mine that I made entirely to be able to show what kind of code I write. The CTO then mentioned that the repo doesn't seem to be very popular and did that in a snarky way.
Sure, it didn't have any starts or any activity on it, but it had many hours of my work in it and it was a perfect indicator of how I coded at the time. Even though I probably dodged a bullet there, them dismissing that entirely because it had no stars was still discouraging.
When interviewer judge your skill on something other then the code itself you have only one thing to do, RUN!!!
Wow. That employer seemed to have missed the point entirely.
All my projects have one star (mine :)) but even tho I'm the only one using it, I try to have the most code coverage and the most comments on the code. You never know if this might be something useful to someone and you best make sure it's easy for another person to use it!
The first thing I check when evaluating a Github project is the commit history. In my opinion, it gives me the best overall impression of the quality the code and the health of the whole project/community. Specifically I'm looking for:
That seems like a very good and thorough approach 👍🏼
If it has more than zero, then they're probably doing it wrong?
I'm not convinced. Hard-to-understand code can be hard to maintain as the codebase grows and evolves. It also becomes hell when there's a bug in code that uses that library, and you have to dig through incomprehensible stack traces.
You have to look at it from the perspective of a recruiter; they don't have the skills to rank developers, nor is there a unified and published metric that you can rank them in general. Even after interviewing someone you might pass up or hire the wrong person, and you're much more qualified. I'm using the general you in this context, not you personally, but we all fall in this category.
The reason they use this metric is because there are no other prominent ones to use and compare other candidates against, so in their view, anything that serves as a proxy for the real value is useful to them, regardless of absolute quality. It's good enough for them; they're just the initial filter.
While I agree that the number of open issues alone isn't very indicative, I think the ratio of open vs closed issues is useful. This can tell you how well the scope of the project is being controlled, how well organised the owner of the repository is, etc.
In your example, VSCode has 3k+ issues of open, but there's a total of 55k issues in their tracker. I start would worry if I saw a tracker with a ratio of around 1 open issue for every 4 closed, depending on maturity of the project.
I agree, that stars should not be used to rank the quality of the project, but they can be really useful. When I published the major release of my recent project (some newsletters, hacker news, etc.), stars on the GitHub helped me a lot.
If your project gets a lot of stars in a short period of time, it is shown on the trending page. This makes your project more visible to other news creators, which also make it more visible to the community.
After super fast growing, it is not soo much important anymore. However, it shows people, that project is noticed by a wider group of programmers.
Unfortunately, people use stars more like something, that tell them if it is worth to read more about a project at all, rather than a clue, that more people already know that, so maybe I should too.
Is that what stars on github are for? I thought they were there to bookmark a repo and not to give of any judgement on a repository.
Stars, downloads, issues, ... they do not say much about quality of code.
Static analysis, unit test code coverage, ... they do say something about the quality of the code.
I also use stars to bookmark repositories I find interesting, usually just based off their Readme description
I use stars as bookmarks and when I review a project later on, if it's not useful or if it becomes part of my toolbox usually I remove the star 🤔
I would add the ability to write a changelog. I get increasingly frustrated by libraries without a changelog, I don't want to read a bunch of diffs to figure out what's changed. It's not a metric of quality per se but it's a signal the author cares about the users and probably a sign there's no rush in the release process.
readable.
A huge percentage of websites run on WordPress, so I'd say the code there is incredibly useful. But the last time I looked at it, I found it quite hard to grok.
But that's just my opinion :).
I work in a startup. we were looking for interns for a technical role and we received few responses. While going through the resume projects they have listed in their CV I noticed most of those projects were just copied from other people on GitHub. The quality of the repo is another thing but sometimes I find it difficult if the project is really genuine, or is it copied from someone and that could be easily find out from looking at the commit history and I do that now
Nope, they shouldn't.
i gave this post a unicorn to mitigate against over-reliance on evaluating based on hearts 🤙
This post was never going to make it to the big leagues either way
I'm not sure it always means good code, but it does usually mean useful or interesting code.
en.wikipedia.org/wiki/Betteridge's...
Most people are not Donald Knuth.