wp-env.json: Pin tt1-blocks dependency to v0.4.3#28741
Merged
Merged
Conversation
ockham
added
the
[Type] Automated Testing
|
Size Change: 0 B Total Size: 1.37 MB ℹ️ View Unchanged
|
Contributor
Author
|
Well, that didn't work 😕 |
Contributor
Author
|
I think we probably need the long-form SHA (unless we get tags), but that's a bit gross -- ideally, we could pass shortened ones, and they'd get auto-expanded for us. |
Contributor
Author
|
Found something here: nodegit/nodegit#569, will try that. |
Contributor
Author
|
Hmm, maybe the problem isn't so much in expanding short SHAs, but in the branch fetching logic (that maybe doesn't work with commit SHAs)? Found this: https://github.com/nodegit/nodegit/blob/7df154a5eb9e93ad933f0a9864f06e2395d32d21/examples/checkout-remote-branch.js -- might give this a try tomorrow. |
Contributor
Author
|
I think the underlying issue is that I've noticed that for the GB release tools, we use a different git library, gutenberg/bin/plugin/lib/git.js Line 4 in 8d1d96d OTOH, nodegit is only used by wp-env's download script.
So I think that he fact that |
Contributor
Author
|
I just discovered #26660. I think it's fair to say that this PR is blocked by that issue. |
ockham
added
the
[Status] Blocked
Contributor
Author
|
@carolinan has kindly tagged |
Contributor
|
Any idea how we'll keep this up to date? |
youknowriad
approved these changes
Feb 9, 2021
Contributor
Author
|
Thanks for approving, Riad! I'm almost surprised this works now -- it never did when I was using a commit SHA to pin instead. I guess it's because of the way we fetch with This means this PR is no longer blocked by #26660. However, it might still be nice to get #28848 in, since
Manually, I'm afraid. That's the price we pay for a stable test environment. The alternative would be to stop relying on an external theme for testing, as discussed in #28692 -- but that has of course other drawbacks; most of all, it's no longer a "real world" theme, so we might lose touch with latests developments and requirements of block template themes more easily. |
ockham
removed
the
[Status] Blocked
Contributor
Author
|
Hmm, |
ockham
force-pushed
the
update/wp-env-tt1-block-theme-dep
branch
from
cd9561f to
703c24a
Compare
Contributor
Author
|
#28884 has been merged, and I've rebased this PR 🤞 |
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
2 participants
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.
Description
Gutenberg's end-to-end tests were broken for a while after TT1 Blocks removed their
front-pagetemplate (WordPress/theme-experiments#179) -- which our e2e tests relied on. This was thankfully fixed on the Gutenberg side by @Addison-Stavlo in #28638.Per discussion with @jeyip, we might be able to avoid a similar situation in the future by pinning our TT1 Blocks dependency to a given version.
See #28692 for more background.
How has this been tested?
Verify that automated tests pass.