Skip to content

Add developers using GitHub actions#11

Merged
bdougie merged 9 commits intobdougie:mainfrom
damaneice:main
Jul 9, 2020
Merged

Add developers using GitHub actions#11
bdougie merged 9 commits intobdougie:mainfrom
damaneice:main

Conversation

@damaneice
Copy link
Contributor

What type of PR is this? (check all applicable)

  • ♻️ Refactor
  • ✨ Feature
  • πŸ› Bug Fix
  • πŸ‘©πŸ½β€πŸ¦± Adding new developer
  • πŸ“ Documentation Update
  • 🚩 Other

Description

This PR adds the ability to automatically add a new developer to the README by creating an issue using the Add handle template.

save ? readme.save! : (puts readme.preview)
end

update_readme(save: true)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

If you need to run this script locally, setting the save flag to false will print the README to the console.

Copy link
Owner

Choose a reason for hiding this comment

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

good to know. I might add that note to the contributing.md

def build_photo_grid(users)
lines = []

users.map{|k, v| k}.each_slice(8) do |slice|
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I just noticed the hardcoded 8 here. I think this needs to be changed. I will take a look at it later

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Never mine I think that is fine

Copy link
Owner

Choose a reason for hiding this comment

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

The 8 is hardcoded because that is the max that a gfm table will show without horizontal scroll

@bdougie
Copy link
Owner

bdougie commented Jul 9, 2020

@damaneice thanks so much for the contribution. Open Source FTW my friend!

@bdougie bdougie merged commit 98500ba into bdougie:main Jul 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants