-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Bump @matrix-org/react-sdk-module-api to 0.0.4 #24642
Conversation
@andybalaam it seems that the pipeline should be fine if restarted, so maybe could try to merge |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like no bumping actually happened given no yarn.lock change
yarn.lock
Outdated
@@ -8110,7 +8103,7 @@ matrix-mock-request@^2.5.0: | |||
|
|||
"matrix-react-sdk@github:matrix-org/matrix-react-sdk#develop": | |||
version "3.67.0" | |||
resolved "https://codeload.github.com/matrix-org/matrix-react-sdk/tar.gz/da2fcdd3462bfcc3ee0033f2d5303942521e1df5" | |||
resolved "https://codeload.github.com/matrix-org/matrix-react-sdk/tar.gz/4f1e5a71dec65258f5295f7d60d2e733391d304c" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks wrong.
Still getting this, something is still not right with the lockfile:
|
I can't understand why I'm still getting:
After my latest commit did a |
Ah, I think it's running against a different commit so it can compare with this one. Given that the main Static Analysis action passed, I think this PR is fine. Merging and keeping an eye in case I'm wrong. |
This gives access to the new widget customisations e.g. matrix-org/matrix-react-sdk#10121
This change is marked as an internal change (Task), so will not be included in the changelog.