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

Need to update version of wordpress env package #377

Closed

Conversation

narenin
Copy link

@narenin narenin commented Sep 23, 2024

Trac ticket: https://buddypress.trac.wordpress.org/ticket/9236

The version is already removed in env package in latest version so updating the same version in package.json

WordPress/gutenberg#63099


This Pull Request is for code review only. Please keep all other discussion in the BuddyPress Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the WordPress Core Handbook for more details.

@narenin
Copy link
Author

narenin commented Sep 23, 2024

@renatonascalves @imath could you please review this.

@imath
Copy link
Contributor

imath commented Sep 23, 2024

Hi @narenin I just launched the Unit tests, if they give good results I'll commit your PR.

@narenin
Copy link
Author

narenin commented Sep 24, 2024

@imath Thanks for review this PR, as I can check now the unit test give good results.

Copy link
Member

@renatonascalves renatonascalves left a comment

Choose a reason for hiding this comment

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

Good catch! ❤️

@renatonascalves
Copy link
Member

@imath Just a reminder that we don't use wp-env in the github actions anymore.

dcavins pushed a commit to dcavins/buddypress-wp-svn that referenced this pull request Oct 7, 2024
This update fixes an issue while trying to run `npm run wp-env start`, where the `version` attribute is not longer available in docker.

Props imath, narenin, and espellcaste.

Fixes #9236
Closes buddypress/buddypress#377

git-svn-id: http://buddypress.svn.wordpress.org/trunk@14030 cdf35c40-ae34-48e0-9cc9-0c9da1808c22
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.

3 participants