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

Bugfix fork detection #377

Merged
merged 6 commits into from Jan 14, 2019

Conversation

Projects
None yet
1 participant
@colszowka
Copy link
Member

colszowka commented Jan 10, 2019

This PR adds a mechanism for detecting "bugfix forks" as part of an effort to improve search result relevance over the next weeks.

There is a huge number of namespaced gems that only have one or maybe a handful of releases, long in the past. Quite frequently they still reference the upstream github repo, so their popularity score is very high for popular gems.

The reason for their existence usually is the trouble that was deploying hotfixed libraries with your app before bundler made including gems from github repos a breeze (thank you bundler, this is so helpful :).

Since you could not simply send a PR with an urgent fix upstream and go with your github branch for a while, many people resorted to the practice of sending a PR and issuing a hotfixed, namespaced gem to github until the PR is reviewed, merged and released upstream.

The idea is to have those projects detected and excluded by default from search results in the future, with a toggle button to make them re-appear.

As it currently stands, this would change the top 50 results when searching the toolbox for authentication like this:

BEFORE AFTER
devise devise
dcu-devise
drfas
loyal_devise
glennr-devise
shingara-devise
ramon-devise
cloudfoundry-devise
ivanvc-devise
devise_omniauth_fb
devise-edge
namxam-devise
upstream-devise
mongoid-devise
rmello-devise
brainsome_devise
af-devise
aihs_devise
graffititracker_devise
devise-no-session
warden warden
omniauth omniauth
mixlib-authentication mixlib-authentication
ruby-hmac ruby-hmac
kth_omniauth
ntlm-http ntlm-http
koala koala
authlogic authlogic
crankharder-authlogic
kschrader-authlogic
cotweet-authlogic
drogus-authlogic
railsware-authlogic
expertiza-authlogic
skippy-authlogic
binarylogic-authlogic
kb-authlogic
jlecour-authlogic
nulogy-authlogic
Empact-authlogic
net-http-digest_auth net-http-digest_auth
rubyntlm rubyntlm
tyler_koala
cotweet_koala
technoweenie/restful-authentication technoweenie/restful-authentication
clearance clearance
thoughtbot-clearance thoughtbot-clearance
gravis-clearance
activerain-clearance
jeffrafter-clearance
nifty-generators
simple_token_authentication
httpauth
devise-two-factor
knock
devise_ldap_authenticatable
spree_auth_devise
omniauth-auth0
authentication-needed-san
authentication-rails
authentication-service
sorcery
rubycas-client
sinatra-authorize
textkey_rest
rubycas-server
signature
two_factor_authentication
pyu-ruby-sasl
sinatra-authentication
pusher-signature
securecompare
devise-token_authenticatable
u2f
google_places
fluent-plugin-secure-forward
rodauth
warden-hmac-authentication
omniauth-authentiq
rails_warden
omniauth-identity
tripletex_api
fresh-auth
refinerycms-authentication
devise_cas_authenticatable
rails-auth
bitbucket_rest_api

@colszowka colszowka changed the title Bugfix for detection Bugfix fork detection Jan 10, 2019

@colszowka colszowka merged commit 2c72558 into master Jan 14, 2019

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@colszowka colszowka deleted the co-fork-detection branch Jan 14, 2019

colszowka added a commit that referenced this pull request Jan 16, 2019

colszowka added a commit that referenced this pull request Jan 16, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment