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

Validate plugin repos exist #53

Closed
wants to merge 18 commits into from
Closed

Conversation

galadril
Copy link
Contributor

@galadril galadril commented Dec 9, 2024

No description provided.

@galadril galadril changed the title Added E-Flux Plugin Added E-Flux and AWTRIX3 Plugins Dec 9, 2024
@galadril
Copy link
Contributor Author

@ycahome can you validate/merge plz?

Add a GitHub Actions workflow to validate the existence of repositories listed in `plugin.py`.

* **.github/workflows/validate.yml**
  - Create a new GitHub Actions workflow file to run the validation script.
  - Define the workflow to run on push and pull request events.
  - Set up Python and install dependencies.
  - Run the validation script.

* **.github/scripts/validate_plugins.py**
  - Create a new script to fetch all plugins data from `plugin.py`.
  - Check if the repositories actually exist and log the results.
Add validation for plugin repositories
@galadril
Copy link
Contributor Author

I noticed that like 7 plugin repositories where giving 404's as they were not existing anymore, so i created this issue to add a GitHub Workflow to parse the plugin data and actually validate if the plugin repos still exist

#54

image

I've cleaned not existing plugins from the list

@galadril galadril changed the title Added E-Flux and AWTRIX3 Plugins Added E-Flux and AWTRIX3 Plugins + validate plugin repos Dec 11, 2024
@galadril
Copy link
Contributor Author

Validate the existings of the repos on PR/commit #54

Copy link
Owner

@ycahome ycahome left a comment

Choose a reason for hiding this comment

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

Added AWTRIX3 Plugin

Copy link
Owner

@ycahome ycahome left a comment

Choose a reason for hiding this comment

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

Added E-Flux Plugin

@ycahome
Copy link
Owner

ycahome commented Dec 19, 2024

There are multiple changes that cannot track. Can you please create a solid pull request with a single commit for each file?

Copy link
Owner

@ycahome ycahome left a comment

Choose a reason for hiding this comment

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

There are multiple changes that cannot track. Can you please create a solid pull request with a single commit for each file?

@galadril galadril changed the title Added E-Flux and AWTRIX3 Plugins + validate plugin repos Validate plugin repos exist Dec 19, 2024
@galadril
Copy link
Contributor Author

Ive made this PR to be only about validating if the repos actually exist
This action/workflow is going to run on PR's so that when a new PR is created, all repo urls are being validated

@galadril
Copy link
Contributor Author

Ill make another one with the 2 new plugins when this is merged. to align the plugin id's ;)

@galadril galadril requested a review from ycahome December 19, 2024 07:28
@galadril galadril closed this Dec 19, 2024
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