Skip to content

Read project info from ADOPTME.md#5

Open
cornelius wants to merge 3 commits intojonobacon:masterfrom
cornelius:ADOPTME
Open

Read project info from ADOPTME.md#5
cornelius wants to merge 3 commits intojonobacon:masterfrom
cornelius:ADOPTME

Conversation

@cornelius
Copy link

If a project is added by adding the URL to an ADOPTME.md file in a repository on GitHub, it's added by parsing the file for info about the contact and the category and taking the rest from the GitHub API.

An example for an ADOPTME.md file is at https://raw.githubusercontent.com/cornelius/kode/master/ADOPTME.md

It would be nice to add a link to the ADOPTME.md in the web interface, so that people can directly go to the detailed info about how to adopt the project.

If a project is added by adding the URL to an `ADOPTME.md` file in
a repository on GitHub, it's added by parsing the file for info
about the contact and the category and taking the rest from the
GitHub API.

An example for an ADOPTME.md file is at
https://raw.githubusercontent.com/cornelius/kode/master/ADOPTME.md

It would be nice to add a link to the ADOPTME.md in the web
interface, so that people can directly go to the detailed info
about how to adopt the project.
@jonobacon
Copy link
Owner

Thanks, @cornelius, for sharing this!

A few things:

  • Can you fix the configparser references to be ConfigParser (it seems it is case dependent).
  • This seems to be mixing a few versions of urllib - can you standardize on one.
  • This uses a github import, but I couldn't figure out a way to install it. It is not installable from pip. How do I get this?

Thanks!

@cornelius
Copy link
Author

I'm not an expert in Python, so I basically fiddled around until I had this sketch of code working. I believe it's the difference between Python 2 and 3, which results in the lower-case names for the modules. I ran it with python3 adopt-queue.py and this worked on my system. Does it work for you if you use Python 3?

I installed the github module with pip install PyGithub.

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