Skip to content

Commit e3c79aa

Browse files
authored
Fix uml permissions (#913)
1 parent ff4eef1 commit e3c79aa

File tree

3 files changed

+7
-5
lines changed

3 files changed

+7
-5
lines changed

.github/workflows/uml.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,15 +10,15 @@ on:
1010
paths:
1111
- "pymc_marketing/**"
1212

13-
permissions: write-all
14-
1513
jobs:
1614
build:
1715
runs-on: ubuntu-latest
18-
16+
permissions: write-all
1917
steps:
2018
- name: Checkout repository
21-
uses: actions/checkout@v2
19+
uses: actions/checkout@v4
20+
with:
21+
ref: ${{ github.head_ref || 'main' }}
2222

2323
- name: Set up Python
2424
uses: actions/setup-python@v2
@@ -43,7 +43,7 @@ jobs:
4343
else
4444
echo "Committing the changes"
4545
git commit -m "Update UML Diagrams"
46-
git push origin HEAD:${GITHUB_HEAD_REF}
46+
git push origin HEAD:${GITHUB_HEAD_REF:-main}
4747
fi
4848
env:
4949
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

docs/source/uml/classes_mmm.png

970 Bytes
Loading

pymc_marketing/mmm/lift_test.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,8 @@
2727

2828

2929
class MissingLiftTestError(Exception):
30+
"""Error when some of the lift tests are not in the model."""
31+
3032
def __init__(self, missing_values: npt.NDArray[np.int_]) -> None:
3133
self.missing_values = missing_values
3234
super().__init__(

0 commit comments

Comments
 (0)