Merged
Conversation
Updated README and action.yml to document and support configuration through environment variables and GitHub Action inputs, in addition to config files. Refactored extract-wp-hooks.php to prioritize environment variables, merge with config file values, and validate required settings. Improved error handling and documentation for configuration options.
Configuration is now only loaded from a JSON config file, simplifying setup and usage. Documentation and scripts have been updated to remove references and logic for environment variable configuration.
Refactors the determination of the GitHub blob URL from a separate workflow step into the config file creation block. This simplifies the workflow and ensures the blob URL is set directly when generating the config file.
Moved the configuration file creation step before cloning the wiki repository and extracting WordPress hooks. This ensures the configuration file is set up prior to repository operations.
The workflow now extracts the wiki directory from the configuration file using jq, and passes it between steps via outputs. This replaces the previous approach of using the input value directly, ensuring consistency with the config file and improving flexibility.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The basic idea is to support configuration through yml inputs, so we don't need to have a separate config file in the repository.
This is mostly AI-generated and I have no idea if this works or how to test it. How did you test the existing version so far?
Changes