forked from openedx/DoneXBlock
-
Notifications
You must be signed in to change notification settings - Fork 1
Merge upstream mods from 2.0.4+ to our course completion branch #1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
bryanlandia
merged 36 commits into
appsembler/course-completion
from
appsembler/upstream-master-merge-course-comp
Jul 15, 2021
Merged
Merge upstream mods from 2.0.4+ to our course completion branch #1
bryanlandia
merged 36 commits into
appsembler/course-completion
from
appsembler/upstream-master-merge-course-comp
Jul 15, 2021
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
python3 compatibility
fix a typo in setup file
* 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.
[BD-6] OEP-18 Compliance
[BD-6] Bump version 2.0.3.
…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`
…40e14 Python Code Cleanup
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>
commits ======= - build(deps): run `make upgrade` - build(deps): add common_constraints file
…ments-5f7e1c7 Python Requirements Update
…ments-d92a0e0 Python Requirements Update
228a85f to
4c66d35
Compare
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 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Merge course completion and scope features into upstream main at 2.0.5-unreleased.