Skip to content

fix: define NEVE_VERSION only when undefined#4550

Open
girishpanchal30 wants to merge 1 commit into
developmentfrom
bugfix/4543
Open

fix: define NEVE_VERSION only when undefined#4550
girishpanchal30 wants to merge 1 commit into
developmentfrom
bugfix/4543

Conversation

@girishpanchal30

Copy link
Copy Markdown
Contributor

Check before Pull Request is ready:

Closes #4543

@girishpanchal30
girishpanchal30 requested a review from Copilot July 21, 2026 10:22
@girishpanchal30 girishpanchal30 added the pr-checklist-skip Allow this Pull Request to skip checklist. label Jul 21, 2026
@pirate-bot pirate-bot added the pr-checklist-complete The Pull Request checklist is complete. (automatic label) label Jul 21, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Conditionally defines NEVE_VERSION, but this does not address the child-theme load-order failure described in #4543.

Changes:

  • Guards the NEVE_VERSION definition with defined().

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread functions.php
Comment on lines +11 to +13
if ( ! defined( 'NEVE_VERSION' ) ) {
define( 'NEVE_VERSION', '4.2.8' );
}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It was fixed with this PR Codeinwp/neve-child#5

@pirate-bot

Copy link
Copy Markdown
Collaborator

Plugin build for 0c24050 is ready 🛎️!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-checklist-complete The Pull Request checklist is complete. (automatic label) pr-checklist-skip Allow this Pull Request to skip checklist.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants