-
Notifications
You must be signed in to change notification settings - Fork 3
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
Write a text for outreach #60
Comments
If we decide to automate some of the above steps, I suggest we use a library to query github, e.g. using any of these https://docs.github.com/en/rest/overview/libraries |
We could try looking for people who have updated or fixed their CITATION.cff file. |
I suggest splitting the "filtering" (basically the steps above) from the "sending a PR" part. Basically the filtering should result in a curated list of repos. |
We (@jspaaks, @abelsiqueira, @fdiblen), decided to remove
|
The final list for filtering:
sorting:
|
We, GT, will first create an issue and ask developers whether they want us to create a PR. Based on the feedback we will create a PR. |
Found another thing to think about: nesting levels, e.g. https://github.com/unwitting-life/cpp-Qt/blob/573f035036485ebb72d6de977ba5bc51d4270385/lib/impl/nlohmann/json/CITATION.cff looks like a checked-in dependency, i.e. not the repo owner's job to correct. |
What do we do with repos where the CITATION.cff file does not pass validation? |
We will do the outreach work here: https://github.com/cffbots/filtering |
Now that we have released version 2.0.0, we could reach out to some folks on GitHub to add the "cffconvert --validate" workflow to their repos via a Pull Request. It's probably a good start to limit this outreach to repos:
CITATION.cff
filesOnce we have a draft text for a Pull Request that adds cffconvert, I suggest we try it out on our colleagues first. If we're happy with their feedback, we can use GitHub search to find other repos that qualify according to the bullet points above, and send them a PR as well.
The text was updated successfully, but these errors were encountered: