Skip to content

Commit 7994966

Browse files
committed
Merge develop into refactor/MVVMMImageBody
Resolve the deleted legacy MImageBody snapshot conflict and preserve deferred URL exposure in ImageBodyViewModel.
2 parents 6474e7a + c4638d1 commit 7994966

303 files changed

Lines changed: 6827 additions & 4894 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/actions/download-verify-element-tarball/action.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ runs:
1111
using: composite
1212
steps:
1313
- name: Download release tarball
14-
uses: robinraju/release-downloader@daf26c55d821e836577a15f77d86ddc078948b05 # v1
14+
uses: robinraju/release-downloader@28fc21f50d76778e7023361aa1f863e717d3d56f # v1
1515
with:
1616
tag: ${{ inputs.tag }}
1717
fileName: element-*.tar.gz*

.github/renovate.json

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,25 +2,39 @@
22
"$schema": "https://docs.renovatebot.com/renovate-schema.json",
33
"extends": ["github>matrix-org/renovate-config-element-web"],
44
"postUpdateOptions": ["pnpmDedupe"],
5+
"stopUpdatingLabel": "X-Blocked",
56
"packageRules": [
67
{
8+
"description": "Group all testcontainers docker digests",
79
"groupName": "testcontainers docker digests",
810
"groupSlug": "testcontainers-docker",
911
"matchDepTypes": ["testcontainers-docker"],
1012
"matchPackageNames": ["*"]
13+
},
14+
{
15+
"description": "Separate updates to overrides from other groups",
16+
"matchDepTypes": ["pnpm.overrides"],
17+
"groupSlug": null
18+
},
19+
{
20+
"description": "Disable any major updates to overrides as this almost always is wrong",
21+
"matchDepTypes": ["pnpm.overrides"],
22+
"matchUpdateTypes": ["major"],
23+
"enabled": false
1124
}
1225
],
1326
"customManagers": [
1427
{
28+
"description": "Update testcontainers docker digests",
1529
"customType": "regex",
1630
"datasourceTemplate": "docker",
1731
"versioningTemplate": "loose",
18-
"description": "Update testcontainers docker digests",
1932
"managerFilePatterns": ["**/testcontainers/*.ts"],
2033
"matchStrings": ["\\s+\"(?<depName>[^@]+):(?<currentValue>[^@]+)@(?<currentDigest>sha256:[a-f0-9]+)\""],
2134
"depTypeTemplate": "testcontainers-docker"
2235
},
2336
{
37+
"description": "Update element-desktop hakDependencies",
2438
"customType": "jsonata",
2539
"managerFilePatterns": ["/(^|/)package\\.json$/"],
2640
"fileFormat": "json",

.github/workflows/build-and-test.yaml

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -206,6 +206,8 @@ jobs:
206206
needs: prepare_ed
207207
name: "Desktop Windows"
208208
uses: ./.github/workflows/build_desktop_windows.yaml
209+
# Skip Windows builds on PRs, as the Linux amd64 build is enough of a smoke test and includes the screenshot tests
210+
if: github.event_name != 'pull_request' || contains(github.event.pull_request.labels.*.name, 'X-Run-All-Tests')
209211
strategy:
210212
matrix:
211213
arch: [x64, ia32, arm64]
@@ -223,10 +225,13 @@ jobs:
223225
arch: [amd64, arm64]
224226
runAllTests:
225227
- ${{ github.event_name != 'pull_request' || contains(github.event.pull_request.labels.*.name, 'X-Run-All-Tests') }}
226-
# We ship static sqlcipher builds, so delegate testing the system builds to the merge queue
227228
exclude:
229+
# We ship static sqlcipher builds, so delegate testing the system builds to the merge queue
228230
- runAllTests: false
229231
sqlcipher: system
232+
# Additionally skip arm64 system builds on PRs, as the amd64 test is enough for a smoke test and includes the screenshot tests
233+
- runAllTests: false
234+
arch: arm64
230235
with:
231236
sqlcipher: ${{ matrix.sqlcipher }}
232237
arch: ${{ matrix.arch }}
@@ -236,6 +241,9 @@ jobs:
236241
needs: prepare_ed
237242
name: "Desktop macOS"
238243
uses: ./.github/workflows/build_desktop_macos.yaml
244+
# Skip macOS builds on PRs, as the Linux amd64 build is enough of a smoke test and includes the screenshot tests
245+
# and we have a very low limit of concurrent macos runners (5) across the Github org.
246+
if: github.event_name != 'pull_request' || contains(github.event.pull_request.labels.*.name, 'X-Run-All-Tests')
239247
with:
240248
blob_report: true
241249

.github/workflows/build_desktop_linux.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,7 @@ jobs:
126126
- name: "Get modified files"
127127
id: changed_files
128128
if: steps.cache.outputs.cache-hit != 'true' && github.event_name == 'pull_request' && github.repository == 'element-hq/element-web'
129-
uses: tj-actions/changed-files@823fcebdb31bb35fdf2229d9f769b400309430d0 # v46
129+
uses: tj-actions/changed-files@9426d40962ed5378910ee2e21d5f8c6fcbf2dd96 # v47
130130
with:
131131
files: |
132132
apps/desktop/dockerbuild/**

.github/workflows/cd.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ jobs:
3131
persist-credentials: false
3232

3333
- name: Install Cosign
34-
uses: sigstore/cosign-installer@398d4b0eeef1380460a10c8013a76f728fb906ac # v3
34+
uses: sigstore/cosign-installer@cad07c2e89fa2edd6e2d7bab4c1aa38e53f76003 # v4.1.1
3535

3636
- name: Set up QEMU
3737
uses: docker/setup-qemu-action@ce360397dd3f832beb865e1373c09c0e9f86d70a # v4

.github/workflows/docker.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
persist-credentials: false
2727

2828
- name: Install Cosign
29-
uses: sigstore/cosign-installer@398d4b0eeef1380460a10c8013a76f728fb906ac # v3
29+
uses: sigstore/cosign-installer@cad07c2e89fa2edd6e2d7bab4c1aa38e53f76003 # v4.1.1
3030
if: github.event_name != 'pull_request'
3131

3232
- name: Set up QEMU

.github/workflows/shared-component-visual-tests-netlify.yaml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,6 @@ jobs:
2525
actions: read
2626
deployments: write
2727
steps:
28-
- name: Install tree
29-
run: "sudo apt-get install -y tree"
30-
3128
- name: Download Diffs
3229
uses: actions/download-artifact@3e5f45b2cfb9172054b4087a40e8e0b5a5461e7c # v8
3330
with:

.github/workflows/shared-component-visual-tests.yaml

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -45,11 +45,12 @@ jobs:
4545
working-directory: packages/shared-components
4646
run: "pnpm test:storybook --run"
4747

48-
# Workaround for vis silently adding new baselines if they didn't exist
49-
# Can be removed once https://github.com/repobuddy/visual-testing/issues/516 is released
50-
- run: |
51-
git add -N .
52-
git diff --exit-code
48+
- name: Detect stale screenshots
49+
run: |
50+
if diff -rq __baselines__ __results__ | grep "^Only in __baselines__"; then
51+
exit 1
52+
fi
53+
working-directory: packages/shared-components/__vis__/linux
5354

5455
- name: Upload received images & diffs
5556
if: always()

.github/workflows/static_analysis.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ jobs:
103103
voip|element_call
104104
error|invalid_json
105105
error|misconfigured
106-
welcome_to_element
106+
welcome|title_element
107107
devtools|settings|elementCallUrl
108108
labs|sliding_sync_description
109109
settings|voip|noise_suppression_description

.github/workflows/update-jitsi.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ jobs:
2727
run: "pnpm vendor:jitsi"
2828

2929
- name: Create Pull Request
30-
uses: peter-evans/create-pull-request@c0f553fe549906ede9cf27b5156039d195d2ece0 # v8
30+
uses: peter-evans/create-pull-request@5f6978faf089d4d20b00c7766989d076bb2fc7f1 # v8
3131
with:
3232
token: ${{ secrets.ELEMENT_BOT_TOKEN }}
3333
branch: actions/jitsi-update

0 commit comments

Comments
 (0)