Skip to content
This repository has been archived by the owner on Apr 3, 2024. It is now read-only.

Commit

Permalink
Merge pull request #2 from eduNEXT/cag/add-maintenance-workflows
Browse files Browse the repository at this point in the history
build: add maintance workflows
  • Loading branch information
bmtcril authored May 5, 2023
2 parents 8a9d84e + a9f269b commit b07e33f
Show file tree
Hide file tree
Showing 11 changed files with 112 additions and 22 deletions.
39 changes: 34 additions & 5 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,10 +1,39 @@
**Description:** Describe in a couple of sentences what this PR adds

**ISSUE:** Link to GitHub issue

**Dependencies:** dependencies on other outstanding PRs, issues, etc.

**Merge deadline:** List merge deadline (if any)

**Installation instructions:** List any non-trivial installation
instructions.

**Testing instructions:**

1. Open page A
2. Do thing B
3. Expect C to happen
4. If D happened instead - check failed.

**Reviewers:**
- [ ] tag reviewer
- [ ] tag reviewer

**Merge checklist:**
Check off if complete *or* not applicable:
- [ ] All reviewers approved
- [ ] CI build is green
- [ ] Version bumped
- [ ] Changelog record added
- [ ] Documentation updated (not only docstrings)
- [ ] Fixup commits are squashed away
- [ ] Unit tests added/updated
- [ ] Manual testing instructions provided
- [ ] Noted any: Concerns, dependencies, migration issues, deadlines, tickets
- [ ] Commits are squashed

**Post merge:**
- [ ] Create a tag
- [ ] Check new version is pushed to PyPI after tag-triggered build is
finished.
- [ ] Delete working branch (if not needed anymore)

**Author concerns:** List any concerns about this PR - inelegant
solutions, hacks, quick-and-dirty implementations, concerns about
migrations, etc.
19 changes: 19 additions & 0 deletions .github/workflows/add-depr-ticket-to-depr-board.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
# Run the workflow that adds new tickets that are either:
# - labelled "DEPR"
# - title starts with "[DEPR]"
# - body starts with "Proposal Date" (this is the first template field)
# to the org-wide DEPR project board

name: Add newly created DEPR issues to the DEPR project board

on:
issues:
types: [opened]

jobs:
routeissue:
uses: openedx/.github/.github/workflows/add-depr-ticket-to-depr-board.yml@master
secrets:
GITHUB_APP_ID: ${{ secrets.GRAPHQL_AUTH_APP_ID }}
GITHUB_APP_PRIVATE_KEY: ${{ secrets.GRAPHQL_AUTH_APP_PEM }}
SLACK_BOT_TOKEN: ${{ secrets.SLACK_ISSUE_BOT_TOKEN }}
19 changes: 19 additions & 0 deletions .github/workflows/add-remove-label-on-comment.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
# This workflow runs when a comment is made on the ticket
# If the comment starts with "label: " it tries to apply
# the label indicated in rest of comment.
# If the comment starts with "remove label: ", it tries
# to remove the indicated label.
# Note: Labels are allowed to have spaces and this script does
# not parse spaces (as often a space is legitimate), so the command
# "label: really long lots of words label" will apply the
# label "really long lots of words label"

name: Allows for the adding and removing of labels via comment

on:
issue_comment:
types: [created]

jobs:
add_remove_labels:
uses: openedx/.github/.github/workflows/add-remove-label-on-comment.yml@master
12 changes: 12 additions & 0 deletions .github/workflows/self-assign-issue.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
# This workflow runs when a comment is made on the ticket
# If the comment starts with "assign me" it assigns the author to the
# ticket (case insensitive)

name: Assign comment author to ticket if they say "assign me"
on:
issue_comment:
types: [created]

jobs:
self_assign_by_comment:
uses: openedx/.github/.github/workflows/self-assign-issue.yml@master
4 changes: 2 additions & 2 deletions .github/workflows/upgrade-python-requirements.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,9 @@ jobs:
with:
branch: ${{ github.event.inputs.branch || 'main' }}
# optional parameters below; fill in if you'd like github or email notifications
user_reviewers: "bmtcril"
user_reviewers: "openedx-event-sink-clickhouse-maintainers"
# team_reviewers: ""
email_address: "[email protected]"
# email_address: ""
# send_success_notification: false
secrets:
requirements_bot_github_token: ${{ secrets.REQUIREMENTS_BOT_GITHUB_TOKEN }}
Expand Down
12 changes: 7 additions & 5 deletions README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,12 @@ Every time you develop something in this repo
Deploying
=========

The Open edX Event Sink Clickhouse component is a django plugin which doesn't
need independent deployment. Therefore, its setup is reasonably
straightforward. First, it needs to be added to your service
requirements, and then it will be installed alongside requirements
of the service.

This plugin will be deployed by default in an OARS Tutor environment. For other
deployments install the library or add it to private requirements of your
virtual environment ( ``requirements/private.txt`` ).
Expand Down Expand Up @@ -122,11 +128,7 @@ Getting Help
Documentation
=============

PLACEHOLDER: Start by going through `the documentation`_. If you need more help see below.

.. _the documentation: https://docs.openedx.org/projects/openedx-event-sink-clickhouse

(TODO: `Set up documentation <https://openedx.atlassian.net/wiki/spaces/DOC/pages/21627535/Publish+Documentation+on+Read+the+Docs>`_)
See `documentation on Read the Docs <https://openedx-event-sink-clickhouse.readthedocs.io/en/latest/>`_.

More Help
=========
Expand Down
4 changes: 2 additions & 2 deletions requirements/base.txt
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ pytz==2023.3
# via
# celery
# django
requests==2.29.0
requests==2.30.0
# via -r requirements/base.in
six==1.16.0
# via click-repl
Expand All @@ -77,7 +77,7 @@ stevedore==5.0.0
# via
# edx-django-utils
# edx-opaque-keys
urllib3==1.26.15
urllib3==2.0.2
# via requests
vine==5.0.0
# via
Expand Down
4 changes: 2 additions & 2 deletions requirements/dev.txt
Original file line number Diff line number Diff line change
Expand Up @@ -266,7 +266,7 @@ pyyaml==6.0
# code-annotations
# edx-i18n-tools
# responses
requests==2.29.0
requests==2.30.0
# via
# -r requirements/quality.txt
# responses
Expand Down Expand Up @@ -328,7 +328,7 @@ typing-extensions==4.5.0
# -r requirements/quality.txt
# astroid
# pylint
urllib3==1.26.15
urllib3==2.0.2
# via
# -r requirements/quality.txt
# requests
Expand Down
13 changes: 11 additions & 2 deletions requirements/doc.txt
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ certifi==2022.12.7
cffi==1.15.1
# via
# -r requirements/test.txt
# cryptography
# pynacl
charset-normalizer==3.1.0
# via
Expand Down Expand Up @@ -71,6 +72,8 @@ coverage[toml]==7.2.5
# via
# -r requirements/test.txt
# pytest-cov
cryptography==40.0.2
# via secretstorage
django==3.2.19
# via
# -c https://raw.githubusercontent.com/edx/edx-lint/master/edx_lint/files/common_constraints.txt
Expand Down Expand Up @@ -121,6 +124,10 @@ iniconfig==2.0.0
# pytest
jaraco-classes==3.2.3
# via keyring
jeepney==0.8.0
# via
# keyring
# secretstorage
jinja2==3.1.2
# via
# -r requirements/test.txt
Expand Down Expand Up @@ -221,7 +228,7 @@ pyyaml==6.0
# responses
readme-renderer==37.3
# via twine
requests==2.29.0
requests==2.30.0
# via
# -r requirements/test.txt
# requests-toolbelt
Expand All @@ -238,6 +245,8 @@ rfc3986==2.0.0
# via twine
rich==13.3.5
# via twine
secretstorage==3.3.3
# via keyring
six==1.16.0
# via
# -r requirements/test.txt
Expand Down Expand Up @@ -300,7 +309,7 @@ typing-extensions==4.5.0
# via
# pydata-sphinx-theme
# rich
urllib3==1.26.15
urllib3==2.0.2
# via
# -r requirements/test.txt
# requests
Expand Down
4 changes: 2 additions & 2 deletions requirements/quality.txt
Original file line number Diff line number Diff line change
Expand Up @@ -200,7 +200,7 @@ pyyaml==6.0
# -r requirements/test.txt
# code-annotations
# responses
requests==2.29.0
requests==2.30.0
# via
# -r requirements/test.txt
# responses
Expand Down Expand Up @@ -243,7 +243,7 @@ typing-extensions==4.5.0
# via
# astroid
# pylint
urllib3==1.26.15
urllib3==2.0.2
# via
# -r requirements/test.txt
# requests
Expand Down
4 changes: 2 additions & 2 deletions requirements/test.txt
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ pyyaml==6.0
# via
# code-annotations
# responses
requests==2.29.0
requests==2.30.0
# via
# -r requirements/base.txt
# responses
Expand Down Expand Up @@ -167,7 +167,7 @@ tomli==2.0.1
# pytest
types-pyyaml==6.0.12.9
# via responses
urllib3==1.26.15
urllib3==2.0.2
# via
# -r requirements/base.txt
# requests
Expand Down

0 comments on commit b07e33f

Please sign in to comment.