Repository navigation
docs: refresh the README - #1
Merged
Merged
Conversation
…cts their own README GitHub shows .github/README.md before the root one, and the scaffolder's File::allFiles() skips dot-directories, so the showcase stays on GitHub and the root README, with placeholders, becomes the generated project's guide.
The rsync now leaves README.md, .github/ and the license files alone (with --delete it removed .github/ and its workflows), the sed loop and leak check skip .github/ and bin/ (they rewrote the template's own tests), and the package.json license is set back after the sync.
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
1 participant
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.
Two READMEs.
.github/README.mdis the GitHub showcase (GitHub shows it before the root one). The scaffolder copies withFile::allFiles(), which skips dot-directories (checked), so it never reaches users. The rootREADME.mdis the generated project's own guide: placeholders thatmake:theme/make:pluginfill in (checked againstbin/replacements.php: no placeholder left), what's inside, the commands, links to pollora.dev. No banner, no org footer.Packaging. The rsync now leaves
/README.md,/.github/,/LICENSEand/license.txtalone. Before this,--deletewould have removed.github/and its workflows, because the dev copies have none. Thepackage.jsonlicense is set back after the sync, and the sed loop and leak check skip.github/.Showcase: banner and badges. Installation via
pollora:make:plugin(and--asset); requirements fixed to PHP 8.4+ / WordPress 7.1+ for a new project (framework PHP 8.3+), instead of PHP 8.2 / WP 6.0. Docs links, footer.Plugin header:
Requires at least: 7.1,Tested up to: 7.1,Requires PHP: 8.4(was 6.0 / 6.9 / 8.2). These follow the skeleton:johnpbloch/wordpress ^7.1, lock on 7.1.2; Symfony 8 in the lock needs PHP 8.4.package-plugin.shreads these three lines before the sync and puts them back after it, because the dev copy still says 6.0 / 8.2. It also forces"license": "MIT"in package.json, since the dev copy says GPL-2.0+.Also: the leak check skips
bin/, like the sed loop. Until now it failed onbin/tests/contract.php, which names the code name on purpose.Checked:
php bin/tests/run.phppasses 17/17. A packaging run from the localpollora-demo-plugincopy succeeds and leaves README, .github, LICENSE and the requirement lines untouched. No tag.