Skip to content
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

Add @TunnelBear and @YorkULibraries #20

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

RichardLitt
Copy link
Member

@khosrow, is there a way for me to automtically generate these templates? It wasn't clear in the contribute guide, and manually adding Markdown files is annoying. I don't want to manually add details for dozens of organizations. :/

@khosrow
Copy link

khosrow commented Jul 12, 2018

@RichardLitt you're looking for @khos2ow ... 😄

@khos2ow
Copy link
Contributor

khos2ow commented Jul 12, 2018

@khosrow but you can become of the maintainers too! 😉

@khos2ow
Copy link
Contributor

khos2ow commented Jul 12, 2018

@RichardLitt hmm, it is in the guide, may be we need to change the wording. https://github.com/opensourcecities/toronto/blob/master/CONTRIBUTING.md#organization

@RichardLitt
Copy link
Member Author

HA! My bad, @khosrow. :) Was using my memory to remember the spelling.

@khos2ow
Copy link
Contributor

khos2ow commented Jul 12, 2018

Essentially you need to execute ./binaries/hugo new organizations/<filename>.md

@RichardLitt RichardLitt added the work-in-progress Work in progress, do not merge label Jul 17, 2018
@RichardLitt RichardLitt self-assigned this Jul 17, 2018
@RichardLitt RichardLitt requested a review from khos2ow August 9, 2018 20:57
@RichardLitt RichardLitt removed their assignment Aug 9, 2018
@RichardLitt RichardLitt removed the work-in-progress Work in progress, do not merge label Aug 9, 2018
@opensourcecities-bot
Copy link
Collaborator

opensourcecities-bot commented Aug 9, 2018

1 Error
🚫 Found 1 link issue

Link issue by awesome_bot

Line Status Link
125 Error https://bentomiso.com/
Connection timed out - connect(2) for "bentomiso.com" port 443

Generated by 🚫 Danger

Copy link
Contributor

@khos2ow khos2ow left a comment

Choose a reason for hiding this comment

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

LGTM, I only had one small comment.

@@ -86,6 +86,7 @@ These projects have been developed mainly in Toronto, or by Torontoians
- [Inclusive Design Institute](https://github.com/inclusive-design) - R&D for inclusiveness and diversity. [Website](https://inclusivedesign.ca/)
- [The Heap](https://github.com/the-heap/) - Cultivated + managed community projects especially welcome to new contributors.
- [Toronto Mesh](https://github.com/tomeshnet) - Community mesh network in Toronto. [Website](https://tomesh.net)
- [yorkulibraries](https://github.com/yorkulibraries) [Website](www.library.yorku.ca)
Copy link
Contributor

Choose a reason for hiding this comment

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

I would say change www.library.yorku.ca to https://www.library.yorku.ca/web/ - as there is a 301 redirect - to prevent awesome_bot complains in the future.

Copy link
Member Author

Choose a reason for hiding this comment

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

I grabbed it from the GitHub profile page. I would just allow 301s on awesome-bot, it's not a blocking error.

Copy link
Contributor

Choose a reason for hiding this comment

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

Great. I would say proceed with this PR and if in following days the issue with https://bentomiso.com/ wasn't resolved we can remove it.

Copy link
Contributor

@khos2ow khos2ow Aug 9, 2018

Choose a reason for hiding this comment

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

Actually if you don't remove the problematic URL, GH pages won't get updated! So you might want to remove it in this PR and ping the owner in an issue.

Copy link
Member Author

Choose a reason for hiding this comment

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

Why won't GH pages get updated?

Copy link
Contributor

Choose a reason for hiding this comment

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

Because it's being updated in Travis after awesome_bot finishes, and if it fails the whole pipeline fails.

Copy link
Contributor

Choose a reason for hiding this comment

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

Compare the last two commits in master branch for example.

Copy link
Member Author

Choose a reason for hiding this comment

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

I'd rather our build didn't depend upon other websites that way. It seems awfully brittle.

@khos2ow
Copy link
Contributor

khos2ow commented Aug 9, 2018

I usually , when awesome_bot fails, run it again after couple of minutes to see if there was a network glitch on failed sites or not and occasionally remove them. But for example this needs to be fixed as it is a legitimate error:

http://hackeryou.com/ redirects to https://hackeryou.com/

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.

4 participants