Skip to content

Commit 5dee196

Browse files
authored
Merge branch 'main' into create-ops-team
2 parents f4f7b01 + a37b909 commit 5dee196

19 files changed

+490
-225
lines changed

.github/ISSUE_TEMPLATE/new-admin.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ body:
99
value: |
1010
Thanks for expressing interest in being an administrator for Django Commons!
1111
> [!NOTE]
12-
> We are currently not accepting new admins. We will be revisiting our admin team around April of 2025. Please send your application though and we will reach out when we re-assess things.
12+
> We revisit our admin team on an annual basis and do not accept new admins on an ad-hoc basis. Please submit this application so that we can reach out when we revisit our admin team. Thank you for your understanding!
1313
- type: checkboxes
1414
id: dsf-member
1515
attributes:

.github/ISSUE_TEMPLATE/new-member.yml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -16,12 +16,6 @@ body:
1616
placeholder: Keep it to 2-4 sentences.
1717
validations:
1818
required: true
19-
- type: textarea
20-
id: which-projects
21-
attributes:
22-
label: Which project(s) are you looking to contribute to?
23-
validations:
24-
required: false
2519
- type: checkboxes
2620
id: general
2721
attributes:

.github/ISSUE_TEMPLATE/transfer-project-in.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,14 @@ body:
5050
options:
5151
- label: Does the project have a test.pypi.org project?
5252
- label: Would you like to test your deployments with uploads to test.pypi.org (you'll need to create the project first)?
53+
- type: input
54+
id: javascript-package-url
55+
attributes:
56+
label: If the project has a JavaScript component published separately to a registry like NPM, what's the URL to that package?
57+
description: Most projects bundle their JavaScript together with the Python package on PyPI -- so this question likely does not apply to you in -- but some projects have a separate JavaScript package that is published to npm. If your project does this, please provide the URL. If not, you can leave this blank.
58+
placeholder: "https://www.npmjs.com/package/your-package-name"
59+
validations:
60+
required: false
5361
- type: markdown
5462
attributes:
5563
value: |

.github/workflows/add_member.yml

Lines changed: 21 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ jobs:
1818
issues: write
1919
env:
2020
ISSUE_NUMBER: ${{ github.event.issue.number }}
21+
ISSUE_USER: ${{ github.event.issue.user.login }}
2122
BRANCH_NAME: "add-user/${{ github.event.issue.number }}"
2223

2324
steps:
@@ -28,35 +29,38 @@ jobs:
2829

2930
- name: Checkout code
3031
uses: actions/checkout@v5
32+
with:
33+
persist-credentials: false
3134

3235
- name: Get username to add
3336
id: get_username
3437
run:
35-
python -c "print('USERNAME='+'${{ github.event.issue.title }}'.split(' - ')[1].strip().lstrip('@'))" >> $GITHUB_ENV
38+
python -c "print('USERNAME='+'${GITHUB_EVENT_ISSUE_TITLE}'.split(' - ')[1].strip().lstrip('@'))" >> $GITHUB_ENV
39+
env:
40+
GITHUB_EVENT_ISSUE_TITLE: ${{ github.event.issue.title }}
3641

3742
- name: Validate add user request
3843
env:
3944
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
40-
ISSUE_USER: ${{ github.event.issue.user.login }}
4145
run: |
4246
# Check whether the user exists
4347
set +e
44-
gh api /users/${{ env.USERNAME }} > /dev/null
48+
gh api /users/${USERNAME} > /dev/null
4549
if [ $? -ne 0 ]; then
46-
gh issue comment ${{ env.ISSUE_NUMBER }} --body "User ${{ env.USERNAME }} does not exist."
50+
gh issue comment ${ISSUE_NUMBER} --body "User ${USERNAME} does not exist."
4751
exit 1
4852
fi
4953
5054
# Check if the username is in the title is the same as the user who opened the issue
51-
if [ "${{ env.USERNAME }}" != "${{ env.ISSUE_USER }}" ]; then
52-
gh issue comment ${{ env.ISSUE_NUMBER }} --body "If you want to add a different user, please create a PR for it"
55+
if [ "${USERNAME}" != "${ISSUE_USER}" ]; then
56+
gh issue comment ${ISSUE_NUMBER} --body "If you want to add a different user, please create a PR for it"
5357
exit 1
5458
fi
5559
5660
# Check if the user is already a member of the org
57-
gh api /orgs/django-commons/members/${{ env.USERNAME }} > /dev/null
61+
gh api /orgs/django-commons/members/${USERNAME} > /dev/null
5862
if [ $? -eq 0 ]; then
59-
gh issue comment ${{ env.ISSUE_NUMBER }} --body "User ${{ env.USERNAME }} is already a member of django-commons."
63+
gh issue comment ${ISSUE_NUMBER} --body "User ${USERNAME} is already a member of django-commons."
6064
exit 1
6165
fi
6266
@@ -66,38 +70,38 @@ jobs:
6670
git config user.email noreply@github.com
6771
6872
- name: Create branch
69-
run: git checkout -b ${{ env.BRANCH_NAME }}
73+
run: git checkout -b ${BRANCH_NAME}
7074

7175
- name: Check if user wants to become a designer
7276
id: check_designer
7377
continue-on-error: true
7478
env:
7579
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
7680
run: |
77-
ADD_TO_DESIGNERS=$(gh issue view ${{ env.ISSUE_NUMBER }} --json body | jq -e '.body | contains("[x] Do you wish to join the designers team?")')
81+
ADD_TO_DESIGNERS=$(gh issue view ${ISSUE_NUMBER} --json body | jq -e '.body | contains("[x] Do you wish to join the designers team?")')
7882
if [ "$ADD_TO_DESIGNERS" = "true" ]; then
79-
python scripts/add_member.py ${{ env.USERNAME }} designers
83+
python scripts/add_member.py ${USERNAME} designers
8084
else
8185
echo "User does not want to join the designers team."
8286
fi
8387
8488
- name: Add user to the list
8589
run: |
86-
python scripts/add_member.py ${{ env.USERNAME }} members
90+
python scripts/add_member.py ${USERNAME} members
8791
8892
- name: Commit changes
8993
run: |
9094
git add terraform/production/org.tfvars
91-
git commit -m "Add ${{ env.USERNAME }} to django-commons"
92-
git push origin ${{ env.BRANCH_NAME }}
95+
git commit -m "Add ${USERNAME} to django-commons"
96+
git push origin ${BRANCH_NAME}
9397
9498
- name: Create pull request
9599
run: |
96100
gh pr create \
97-
--title "Add ${{ env.USERNAME }} to django-commons" \
98-
--body "Fix #${{ env.ISSUE_NUMBER }}" \
101+
--title "Add ${USERNAME} to django-commons" \
102+
--body "Fix #${ISSUE_NUMBER}\n[New Member Playbook](https://github.com/django-commons/controls?tab=readme-ov-file#new-member-playbook)" \
99103
--base main \
100-
--head ${{ env.BRANCH_NAME }} \
104+
--head ${BRANCH_NAME} \
101105
--label "New member"
102106
env:
103107
GITHUB_TOKEN: ${{ secrets.TERRAFORM_MANAGEMENT_GITHUB_TOKEN }}

.github/workflows/apply.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,8 @@ jobs:
2222
steps:
2323
- name: Checkout code
2424
uses: actions/checkout@v4
25+
with:
26+
persist-credentials: false
2527

2628
- name: terraform apply
2729
# v1.44.0
@@ -40,7 +42,8 @@ jobs:
4042
4143
- name: Commit changes
4244
if: ${{ always() }}
43-
uses: devops-infra/action-commit-push@v0.10.0
45+
# v0.10.0
46+
uses: devops-infra/action-commit-push@b8c990ac36bac67f71133ad7ec3da1d7abf4d57e
4447
with:
4548
github_token: "${{ secrets.TERRAFORM_MANAGEMENT_GITHUB_TOKEN }}"
4649
commit_prefix: "[AUTO]"

.github/workflows/member-verification.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,8 @@ jobs:
1313
steps:
1414
- name: Checkout code
1515
uses: actions/checkout@v4
16+
with:
17+
persist-credentials: false
1618

1719
- name: Check PR title format
1820
id: check-title
@@ -85,4 +87,4 @@ jobs:
8587
repo: context.repo.repo,
8688
issue_number: context.issue.number,
8789
labels: ['verification-needed']
88-
});
90+
});

.github/workflows/plan.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ jobs:
2121
uses: actions/checkout@v4
2222
with:
2323
ref: "${{ github.event.pull_request.head.ref }}"
24+
persist-credentials: false
2425

2526

2627
- name: terraform fmt
@@ -29,7 +30,8 @@ jobs:
2930
path: "terraform"
3031

3132
- name: Commit changes
32-
uses: devops-infra/action-commit-push@v0.9.2
33+
# v0.9.2
34+
uses: devops-infra/action-commit-push@be5ba37125c79eb0016cbd8cc385f0aa160538c5
3335
with:
3436
github_token: "${{ secrets.GITHUB_TOKEN }}"
3537
commit_prefix: "[AUTO]"
@@ -48,6 +50,8 @@ jobs:
4850
steps:
4951
- name: Checkout code
5052
uses: actions/checkout@v4
53+
with:
54+
persist-credentials: false
5155

5256
- name: terraform plan
5357
# v1.44.0

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,4 +101,4 @@ Any package is welcome to join Django Commons so long as it has tangible benefit
101101

102102
### Logo
103103

104-
Vectors and icons by [Raisul Hadi](https://dribbble.com/Broc_Simp?ref=svgrepo.com) in CC Attribution License via [SVG Repo](https://www.svgrepo.com/).
104+
Logo was designed by [Violette Naa Adoley Allotey](https://www.linkedin.com/in/violette-naa-adoley-allotey/)

assets/django-commons-logo-dark.svg

Lines changed: 1 addition & 0 deletions
Loading

assets/django-commons-logo-full.svg

Lines changed: 145 additions & 0 deletions
Loading

0 commit comments

Comments
 (0)