Skip to content

Conversation

@bryanlandia
Copy link

@bryanlandia bryanlandia commented Jul 15, 2021

Merge course completion and scope features into upstream main at 2.0.5-unreleased.

noraiz-anwar and others added 30 commits October 22, 2019 15:27
* Removing py2 vs py3 diff

* Adding version num to __init__.py
…deprecation

[PROD-1440] - Handle fragment DeprecationWarning to reduce logs.
Create requirements folder.
create pip_tools and base requirements files.
Add Makefile.
Add makefile upgrade command.
Include requirements files generated using upgrade command.
Update openedx.yaml OEPs list.
…b, which ran

python -m edx_repo_tools.modernize_openedx_yaml --path=openedx.yaml
The following packages were installed: git+https://github.com/edx/repo-tools.git@4ac2ffde02b6df4bf633f7b88966137d71233bb1.
This pull request was generated by the cleanup-python-code Jenkins job, which ran
```
make upgrade;find . -type f -name '*.py' | while read fname; do pyupgrade --exit-zero-even-if-changed --keep-percent-format --py3-plus --py36-plus --py38-plus "$fname"; done
```

The following packages were installed:
`pyupgrade,git+https://github.com/edx/repo-tools.git@c05453b733602ba7a2224ffbbf58a8d0903b62d8`
Release on pypi through github actions
…ements

feat: Added upgrade-python-requirements workflow
s/teaching-and-learning@edx.opsgenie.net/community-engineering@edx.opsgenie.net/
build: update escalation email address
Background
==========
As part of our Squad-based ownership, we should stay on top of what
happens in these repositories. However, due to the number of
repositories (and subsequently pull requests) across the edX ecosystem,
it is challenging to stay on top of notifications, separating the
'signal' from the 'noise'. Email filters can go a long way to taming
Inbox notifications, but this is manual and requires maintenance as
Squad ownership changes. It also fails to account for Github-specific behavior.

Proposal
========
By leveraging Github support for `CODEOWNERS` files [1],
we can ensure that our team is at least CCed explicitly, here,
in the form a requested review. This request is just that, a request,
not a requirement; we are not enacting any new merge requirements
at this time.

- [1] https://docs.github.com/en/github/creating-cloning-and-archiving-repositories/about-code-owners
The Community Engineering team has agreed that we would only like to get tagged on Python upgrade notifications, not paged in Ops Genie. This PR removes OG notification and also removes the team-reviewers as that was not working; we are moving to being tagged in the CODEOWNERS file.
Bumps [pyyaml](https://github.com/yaml/pyyaml) from 5.3.1 to 5.4.
- [Release notes](https://github.com/yaml/pyyaml/releases)
- [Changelog](https://github.com/yaml/pyyaml/blob/master/CHANGES)
- [Commits](yaml/pyyaml@5.3.1...5.4)

---
updated-dependencies:
- dependency-name: pyyaml
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [lxml](https://github.com/lxml/lxml) from 4.6.2 to 4.6.3.
- [Release notes](https://github.com/lxml/lxml/releases)
- [Changelog](https://github.com/lxml/lxml/blob/master/CHANGES.txt)
- [Commits](lxml/lxml@lxml-4.6.2...lxml-4.6.3)

---
updated-dependencies:
- dependency-name: lxml
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
@bryanlandia bryanlandia force-pushed the appsembler/upstream-master-merge-course-comp branch from 228a85f to 4c66d35 Compare July 15, 2021 00:08
@bryanlandia bryanlandia marked this pull request as ready for review July 15, 2021 00:09
@bryanlandia bryanlandia marked this pull request as draft July 15, 2021 00:16
@bryanlandia
Copy link
Author

Need to first merge https://github.com/appsembler/edx-configs/pull/1157 so we don't accidentally deploy Juniper changes to ISC Ficus prod or staging

@bryanlandia bryanlandia changed the title Appsembler/upstream master merge course comp Merge upstream mods from 2.0.4+ to our course completion branch Jul 15, 2021
@bryanlandia bryanlandia marked this pull request as ready for review July 15, 2021 23:47
@bryanlandia bryanlandia merged commit 64ac0e3 into appsembler/course-completion Jul 15, 2021
@bryanlandia bryanlandia deleted the appsembler/upstream-master-merge-course-comp branch July 15, 2021 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.