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

feat: more automated awesome-ipfs PRs #428

Closed
wants to merge 26 commits into from
Closed

Conversation

SgtPooki
Copy link
Member

@SgtPooki SgtPooki commented Sep 14, 2022

What kind of PR is this?

Select only one to speed up review/approval; i.e., don't lump an addition and a removal into a single PR.

  • [N/A] Adding something new to awesome-ipfs
  • [N/A] Editing something already listed on awesome-ipfs
  • [N/A] Removing something from awesome-ipfs
  • Something else (if so, please explain in the "Additional details" section below)

Pre-submit checklist

Please confirm ALL of the following before submitting your PR.

Additional details

Is there anything else we should know about this PR?

This PR is a part of addressing concerns raised in #420 and elsewhere, and adds the following features:

  • Automatically update README.md for users. All they need to do to contribute is modify the data/** files.
  • Runs awesome_bot via github actions instead of via circleci
  • Posts comment to PRs giving insight into what was changed.
    • If .github/** was changed, a regular message will be displayed.
    • If data/** was changed
      • A message will be displayed indicating the YAML data has been edited.
      • If there are broken links, danger-js will fail the PR
    • If src/** was changed, a warning will be displayed
    • If scripts/** was changed, a warning will be displayed

See SgtPooki#2 for an example of what these changes do. You can look at the edit history for SgtPooki#2 (comment) to see the different types of messages.

@SgtPooki
Copy link
Member Author

replaced by #429

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