Skip to content
This repository has been archived by the owner on Apr 29, 2020. It is now read-only.

Lead/Notetaker Fields not filled in #12

Closed
dgrisham opened this issue Jan 8, 2018 · 7 comments
Closed

Lead/Notetaker Fields not filled in #12

dgrisham opened this issue Jan 8, 2018 · 7 comments

Comments

@dgrisham
Copy link

dgrisham commented Jan 8, 2018

The bot doesn't seem to be filling in the lead/notetaker fields in the all-hands template. This appears to have been happening since early December (<-- may have also happened the week before that issue).

@RichardLitt
Copy link
Contributor

Is this bot still active?

@dgrisham
Copy link
Author

dgrisham commented Jan 8, 2018

I believe so, unless a different one is being used now. @flyingzumwalt?

@RichardLitt
Copy link
Contributor

It grabs those from the README specified. https://github.com/ipfs/node-github-issue-bot/blob/master/index.js#L25

The issue is probably here: https://github.com/ipfs/pm/commits/master somewhere.

@RichardLitt
Copy link
Contributor

Hmm, doesn't seem to be a README issue, actually. Not sure why that's happening, then.

@dgrisham
Copy link
Author

dgrisham commented Jan 8, 2018

Hm, yeah it looks like the only commit that might've affected the README before this starting happening was here, which is pretty benign.

@RichardLitt
Copy link
Contributor

Tests might help. I'd be happy to look over any if you want to help PR stuff? I don't actively maintain this at the moment, unfortunately, and don't have time to check it out myself right now.

@dgrisham
Copy link
Author

dgrisham commented Jan 8, 2018

Ohh, got it. The last notetaker was edited in, but wasn't on the list in the README. As a result, this case executed. Seems better (at least for now) to fix this on the README and not in this repo. Thanks for the guidance!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants