Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix github url strings (org edx -> openedx) #63

Merged
merged 3 commits into from
Sep 14, 2022
Merged

Fix github url strings (org edx -> openedx) #63

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
332a1d8
fix: fix github url strings (org edx -> openedx)
sarina Sep 10, 2022
fe791cb
fix: update path to .github workflows to read from openedx org
sarina Sep 10, 2022
5b1f915
chore: run edx_lint update with the current version of the repo.
sarina Sep 14, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/commitlint.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,4 @@ on:

jobs:
commitlint:
uses: edx/.github/.github/workflows/commitlint.yml@master
uses: openedx/.github/.github/workflows/commitlint.yml@master
2 changes: 1 addition & 1 deletion .github/workflows/upgrade-python-requirements.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ jobs:

- name: setup testeng-ci
run: |
git clone https://github.com/edx/testeng-ci.git
git clone https://github.com/openedx/testeng-ci.git
cd $GITHUB_WORKSPACE/testeng-ci
pip install -r requirements/base.txt
- name: create pull request
Expand Down
4 changes: 2 additions & 2 deletions README.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,5 +12,5 @@ License
The code in this repository is licensed under version 3 of the AGPL unless otherwise noted. Please see the `LICENSE`_ file for details.


.. _edx-platform: https://github.com/edx/edx-platform
.. _LICENSE: https://github.com/edx/openedx-calc/blob/master/LICENSE
.. _edx-platform: https://github.com/openedx/edx-platform
.. _LICENSE: https://github.com/openedx/openedx-calc/blob/master/LICENSE
12 changes: 6 additions & 6 deletions pylintrc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
# ** DO NOT EDIT THIS FILE **
# ***************************
#
# This file was generated by edx-lint: https://github.com/edx/edx-lint
# This file was generated by edx-lint: https://github.com/openedx/edx-lint
#
# If you want to change this file, you have two choices, depending on whether
# you want to make a local change that applies only to this repo, or whether
Expand All @@ -28,7 +28,7 @@
# CENTRAL CHANGE:
#
# 1. Edit the pylintrc file in the edx-lint repo at
# https://github.com/edx/edx-lint/blob/master/edx_lint/files/pylintrc
# https://github.com/openedx/edx-lint/blob/master/edx_lint/files/pylintrc
#
# 2. install the updated version of edx-lint (in edx-lint):
#
Expand Down Expand Up @@ -64,10 +64,10 @@
# SERIOUSLY.
#
# ------------------------------
# Generated by edx-lint version: 5.2.4
# Generated by edx-lint version: 5.2.5
# ------------------------------
[MASTER]
ignore =
ignore = migrations
persistent = yes
load-plugins = edx_lint.pylint,pylint_django,pylint_celery

Expand Down Expand Up @@ -257,7 +257,6 @@ enable =
deprecated-pragma,
unrecognized-inline-option,
useless-suppression,

disable =
bad-indentation,
consider-using-f-string,
Expand Down Expand Up @@ -287,6 +286,7 @@ disable =
illegal-waffle-usage,

logging-fstring-interpolation,
no-self-use,
django-not-available,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was the hand-edited line that is now missing from pylintrc, and is causing the linting failures.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah thanks, I'll add that to _tweaks


[REPORTS]
Expand Down Expand Up @@ -384,4 +384,4 @@ int-import-graph =
[EXCEPTIONS]
overgeneral-exceptions = Exception

# fad4071fe38e655a180214bd3f6bce3ffa41b2ed
# 0fc0d3e6714dbf532960c58d542877f3741cfdc8
9 changes: 9 additions & 0 deletions pylintrc_tweaks
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
# pylintrc tweaks for use with edx_lint.
[MASTER]
ignore = migrations
load-plugins = edx_lint.pylint,pylint_django,pylint_celery

[MESSAGES CONTROL]
disable+ =
no-self-use,
django-not-available,
2 changes: 1 addition & 1 deletion setup.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ def get_version(*file_paths):
long_description_content_type="text/x-rst",
author='edX',
author_email='oscm@edx.org',
url='https://github.com/edx/openedx-calc',
url='https://github.com/openedx/openedx-calc',
packages=[
'calc',
"symmath"
Expand Down