Skip to content

Updated reference implementation (initial value, description bonus)#229

Merged
lenucksi merged 1 commit intoInnerSourceCommons:masterfrom
Michadelic:patch-4
Oct 3, 2020
Merged

Updated reference implementation (initial value, description bonus)#229
lenucksi merged 1 commit intoInnerSourceCommons:masterfrom
Michadelic:patch-4

Conversation

@Michadelic
Copy link
Copy Markdown
Contributor

  • starts score calculation with 50 instead of 1 to give active projects with low GitHub stats (forks, stars, watches) a little advantage
  • gives projects with a meaningful description (>30 chars) a small bonus of 50 to be listed higher than similar projects with no description

* starts score calculation with 50 instead of 1 to give active projects with low GitHub stats (forks, stars, watches) a little advantage
* gives projects with a meaningful description (>30 chars) a small bonus of 50 to be listed higher than similar projects with no description
Copy link
Copy Markdown
Member

@spier spier left a comment

Choose a reason for hiding this comment

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

Nothing to object with, give that this is how you do it 👍

For me the comments in the pseudocode are equally if not more interesting than the code itself as they describe your intention.

Great stuff!

@lenucksi lenucksi added the 📖 Type - Content Work Working on contents is the main focus of this issue / PR label Oct 3, 2020
Copy link
Copy Markdown
Member

@lenucksi lenucksi left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM

@lenucksi lenucksi merged commit 273507f into InnerSourceCommons:master Oct 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📖 Type - Content Work Working on contents is the main focus of this issue / PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants