Skip to content

Commit ee1c44f

Browse files
Merge branch 'master' into ktyagi/mktg
2 parents fa582da + 190a8b8 commit ee1c44f

File tree

378 files changed

+9242
-7771
lines changed

Some content is hidden

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

378 files changed

+9242
-7771
lines changed

.github/CODEOWNERS

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,11 @@ openedx/core/djangoapps/user_api/ @openedx/com
2121
openedx/core/djangoapps/user_authn/ @openedx/committers-edx-platform-2u-infinity
2222
openedx/core/djangoapps/verified_track_content/ @openedx/committers-edx-platform-2u-infinity
2323
openedx/features/course_experience/
24-
xmodule/
24+
# The Aximprovements team is working on extracting all built-in XBlocks
25+
# to the external repository (xblocks-contrib). They need to be notified
26+
# about any changes within xmodule to stay aligned with this effort.
27+
# Ticket: https://github.com/openedx/edx-platform/issues/34827
28+
xmodule/ @farhan @irtazaakram @salman2013
2529

2630
# Core Extensions
2731
lms/djangoapps/discussion/

.github/workflows/check-consistent-dependencies.yml

Lines changed: 19 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ name: Consistent Python dependencies
77

88
on:
99
pull_request:
10+
merge_group:
1011

1112
defaults:
1213
run:
@@ -18,26 +19,31 @@ jobs:
1819
runs-on: ubuntu-24.04
1920

2021
steps:
22+
# Always checkout the code because we don't always have a PR url.
23+
- uses: actions/checkout@v5
24+
2125
# Only run remaining steps if there are changes to requirements/**
26+
# We do this instead of using path based short-circuiting.
27+
# see https://stackoverflow.com/questions/77996177/how-can-i-handle-a-required-check-that-isnt-always-triggered
28+
# for some more details.
2229
- name: "Decide whether to short-circuit"
23-
env:
24-
GH_TOKEN: "${{ github.token }}"
25-
PR_URL: "${{ github.event.pull_request.html_url }}"
2630
run: |
27-
paths=$(gh pr diff "$PR_URL" --name-only)
28-
echo $'Paths touched in PR:\n'"$paths"
31+
if [[ "${{ github.event_name }}" == "pull_request" ]]; then
32+
BASE_SHA="${{ github.event.pull_request.base.sha }}"
33+
else
34+
BASE_SHA="${{ github.event.merge_group.base_sha }}"
35+
fi
36+
37+
# Fetch the base sha so we can compare to it. It's not checked out by
38+
# default.
39+
git fetch origin "$BASE_SHA"
2940
3041
# The ^"? is because git may quote weird file paths
31-
matched="$(echo "$paths" | grep -P '^"?((requirements/)|(scripts/.*?/requirements/))' || true)"
32-
echo $'Relevant paths:\n'"$matched"
33-
if [[ -n "$matched" ]]; then
34-
echo "RELEVANT=true" >> "$GITHUB_ENV"
42+
if git diff --name-only "$BASE_SHA" | grep -P '^"?((requirements/)|(scripts/.*?/requirements/))'; then
43+
echo "RELEVANT=true" >> "$GITHUB_ENV"
3544
fi
3645
37-
- uses: actions/checkout@v5
38-
if: ${{ env.RELEVANT == 'true' }}
39-
40-
- uses: actions/setup-python@v5
46+
- uses: actions/setup-python@v6
4147
if: ${{ env.RELEVANT == 'true' }}
4248
with:
4349
python-version: '3.11'

.github/workflows/check-for-tutorial-prs.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
uses: actions/checkout@v5
2727

2828
- name: Comment PR
29-
uses: thollander/actions-comment-pull-request@v2
29+
uses: thollander/actions-comment-pull-request@v3
3030
with:
3131
message: |
3232
Thank you for your pull request! Congratulations on completing the Open edX tutorial! A team member will be by to take a look shortly.

.github/workflows/check_python_dependencies.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ name: Check Python Dependencies
22

33
on:
44
pull_request:
5+
merge_group:
56

67
jobs:
78
check_dependencies:
@@ -16,7 +17,7 @@ jobs:
1617
uses: actions/checkout@v5
1718

1819
- name: Set up Python
19-
uses: actions/setup-python@v5
20+
uses: actions/setup-python@v6
2021
with:
2122
python-version: ${{ matrix.python-version }}
2223

.github/workflows/ci-static-analysis.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
name: Static analysis
22

3-
on: pull_request
3+
on:
4+
pull_request:
5+
merge_group:
46

57
jobs:
68
tests:
@@ -15,7 +17,7 @@ jobs:
1517
steps:
1618
- uses: actions/checkout@v5
1719
- name: Set up Python
18-
uses: actions/setup-python@v5
20+
uses: actions/setup-python@v6
1921
with:
2022
python-version: ${{ matrix.python-version }}
2123

.github/workflows/commitlint.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,8 @@
33
name: Lint Commit Messages
44

55
on:
6-
- pull_request
6+
pull_request:
7+
merge_group:
78

89
jobs:
910
commitlint:

.github/workflows/compile-python-requirements.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ jobs:
2424
ref: "${{ inputs.branch }}"
2525

2626
- name: Set up Python environment
27-
uses: actions/setup-python@v5
27+
uses: actions/setup-python@v6
2828
with:
2929
python-version: "3.11"
3030

.github/workflows/js-tests.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ name: Javascript tests
22

33
on:
44
pull_request:
5+
merge_group:
56
push:
67
branches:
78
- master
@@ -23,7 +24,7 @@ jobs:
2324
run: git fetch --depth=1 origin master
2425

2526
- name: Setup Node
26-
uses: actions/setup-node@v4
27+
uses: actions/setup-node@v5
2728
with:
2829
node-version: ${{ matrix.node-version }}
2930
cache: 'npm'
@@ -43,7 +44,7 @@ jobs:
4344
run: sudo apt-get update && sudo apt-get install libxmlsec1-dev ubuntu-restricted-extras xvfb
4445

4546
- name: Setup Python
46-
uses: actions/setup-python@v5
47+
uses: actions/setup-python@v6
4748
with:
4849
python-version: ${{ matrix.python-version }}
4950

.github/workflows/lint-imports.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ name: Lint Python Imports
22

33
on:
44
pull_request:
5+
merge_group:
56
push:
67
branches:
78
- master
@@ -16,7 +17,7 @@ jobs:
1617
uses: actions/checkout@v5
1718

1819
- name: Set up Python
19-
uses: actions/setup-python@v5
20+
uses: actions/setup-python@v6
2021
with:
2122
python-version: "3.11"
2223

.github/workflows/lockfileversion-check.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ on:
77
branches:
88
- master
99
pull_request:
10+
merge_group:
1011

1112
jobs:
1213
version-check:

0 commit comments

Comments
 (0)