Repository navigation
Update sources.json - #2700
Update sources.json#2700ibrahimsif6-netizen wants to merge 10 commits into
Conversation
[Low impact] Updates a URL in plugin configuration data. Restore the comma and increase the Novels Paradise version before merging. FindingsSummaryChanges the Novels Paradise base URL to
Reviews (1) · Last reviewed commit: "Update sources.json" · Reviewed by Greptile |
| { | ||
| "id": "novelsparadise", | ||
| "sourceSite": "https://novelsparadise.site/", | ||
| "sourceSite": "https://novelsparadise.site/np-light/" |
There was a problem hiding this comment.
LightNovelWP plugins fail to generate
The changed sourceSite line is missing its trailing comma, making sources.json invalid JSON. generator.js imports this file before generating any plugins, so the entire LightNovelWP family is skipped. After build:full cleans the generated files, none of those plugins are regenerated. Restore the comma.
| "sourceSite": "https://novelsparadise.site/np-light/" | |
| "sourceSite": "https://novelsparadise.site/np-light/", |
Knowledge Base Used: Multi-source plugin generation
| { | ||
| "id": "novelsparadise", | ||
| "sourceSite": "https://novelsparadise.site/", | ||
| "sourceSite": "https://novelsparadise.site/np-light/" |
There was a problem hiding this comment.
The URL changes without increasing options.versionIncrements, so Novels Paradise still builds as 1.1.12. Even after fixing the comma, build-plugin-manifest.js skips this changed plugin under --only-new when the existing version is the same. Add versionIncrements: 1 so the URL fix has a newer version, as the family’s author guide requires for URL changes.
Knowledge Base Used: Multi-source plugin generation
Checklist
type(scope): description(e.g.feat(<generator>): add new source)