Skip to content

Conversation

@sjinks
Copy link
Member

@sjinks sjinks commented Feb 20, 2025

The idea is to write $WP_GIT_REF into wp-content/.version. This will allow us to skip rsync if it is not needed, improving the startup time (esp. on Windows).

Because of Automattic/vip-cli#2240, we had to use a checksum-based comparison, meaning we had to read all files at the source and the destination (instead of just directory metadata).

The idea behind this PR is to skip the rsync step altogether if our WordPress version and target WordPress versions are the same.

@sjinks sjinks force-pushed the update/wordpress branch 2 times, most recently from 55b11d2 to 2238054 Compare February 24, 2025 11:48
@github-actions
Copy link

This pull request has been marked stale because it has been open for 60 days with no activity. If there is no activity within 7 days, it will be closed.

This is an automation to keep pull requests manageable and actionable and is not a comment on the quality of this pull request nor on the work done so far. Closed PRs are still valuable to the project and their branches are preserved.

@github-actions
Copy link

This pull request has been marked stale because it has been open for 60 days with no activity. If there is no activity within 7 days, it will be closed.

This is an automation to keep pull requests manageable and actionable and is not a comment on the quality of this pull request nor on the work done so far. Closed PRs are still valuable to the project and their branches are preserved.

@github-actions
Copy link

github-actions bot commented Oct 1, 2025

This pull request has been marked stale because it has been open for 60 days with no activity. If there is no activity within 7 days, it will be closed.

This is an automation to keep pull requests manageable and actionable and is not a comment on the quality of this pull request nor on the work done so far. Closed PRs are still valuable to the project and their branches are preserved.

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.

2 participants