Skip to content

Commit

Permalink
fix: requirements to make forum work with edx-platform (#121)
Browse files Browse the repository at this point in the history
* fix: pin pylint to the same version as that of edx
As updating the pylint version to latest fails edx CI and
If pinning it to the same version as it(forum) has before, it fails compiling the requirements in edx-platofrm
So, adding the same constraint for pylint as edx-platform has for it. i.e. `pylint<2.16.0`

* fix: upgrade requirements
- added `too-many-positional-arguments` disable list in pylintrc to fix these warning:https://github.com/edly-io/forum/actions/runs/11513603527/job/32050616644

---------

Co-authored-by: Muhammad Faraz  Maqsood <[email protected]>
Co-authored-by: Muhammad Faraz  Maqsood <[email protected]>
  • Loading branch information
3 people authored Oct 25, 2024
1 parent 6f794b8 commit 38183d2
Show file tree
Hide file tree
Showing 8 changed files with 10 additions and 80 deletions.
1 change: 1 addition & 0 deletions pylintrc
Original file line number Diff line number Diff line change
Expand Up @@ -293,6 +293,7 @@ disable =
bad-option-value,
missing-timeout,
useless-suppression,
too-many-positional-arguments,

[REPORTS]
output-format = text
Expand Down
1 change: 0 additions & 1 deletion requirements/base.in
Original file line number Diff line number Diff line change
Expand Up @@ -10,4 +10,3 @@ requests
pymongo
elasticsearch
mysqlclient
pylint==3.2.7
14 changes: 0 additions & 14 deletions requirements/base.txt
Original file line number Diff line number Diff line change
Expand Up @@ -6,16 +6,12 @@
#
asgiref==3.8.1
# via django
astroid==3.2.4
# via pylint
certifi==2024.8.30
# via
# elasticsearch
# requests
charset-normalizer==3.4.0
# via requests
dill==0.3.9
# via pylint
django==4.2.16
# via
# -c https://raw.githubusercontent.com/edx/edx-lint/master/edx_lint/files/common_constraints.txt
Expand All @@ -31,26 +27,16 @@ elasticsearch==7.13.4
# -r requirements/base.in
idna==3.10
# via requests
isort==5.13.2
# via pylint
mccabe==0.7.0
# via pylint
mysqlclient==2.2.5
# via -r requirements/base.in
openedx-atlas==0.6.2
# via -r requirements/base.in
platformdirs==4.3.6
# via pylint
pylint==3.2.7
# via -r requirements/base.in
pymongo==4.10.1
# via -r requirements/base.in
requests==2.32.3
# via -r requirements/base.in
sqlparse==0.5.1
# via django
tomlkit==0.13.2
# via pylint
urllib3==1.26.20
# via
# elasticsearch
Expand Down
4 changes: 2 additions & 2 deletions requirements/ci.txt
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ asgiref==3.8.1
# -r requirements/quality.txt
# django
# django-stubs
astroid==3.2.4
astroid==3.3.5
# via
# -r requirements/quality.txt
# pylint
Expand Down Expand Up @@ -167,7 +167,7 @@ pycodestyle==2.12.1
# via -r requirements/quality.txt
pydocstyle==6.3.0
# via -r requirements/quality.txt
pylint==3.2.7
pylint==3.3.1
# via
# -r requirements/quality.txt
# edx-lint
Expand Down
4 changes: 2 additions & 2 deletions requirements/dev.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ asgiref==3.8.1
# -r requirements/quality.txt
# django
# django-stubs
astroid==3.2.4
astroid==3.3.5
# via
# -r requirements/ci.txt
# -r requirements/quality.txt
Expand Down Expand Up @@ -249,7 +249,7 @@ pydocstyle==6.3.0
# -r requirements/quality.txt
pygments==2.18.0
# via diff-cover
pylint==3.2.7
pylint==3.3.1
# via
# -r requirements/ci.txt
# -r requirements/quality.txt
Expand Down
23 changes: 0 additions & 23 deletions requirements/doc.txt
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,6 @@ asgiref==3.8.1
# via
# -r requirements/test.txt
# django
astroid==3.2.4
# via
# -r requirements/test.txt
# pylint
babel==2.16.0
# via
# pydata-sphinx-theme
Expand Down Expand Up @@ -57,10 +53,6 @@ coverage[toml]==7.6.4
# via
# -r requirements/test.txt
# pytest-cov
dill==0.3.9
# via
# -r requirements/test.txt
# pylint
distlib==0.3.9
# via
# -r requirements/test.txt
Expand Down Expand Up @@ -110,10 +102,6 @@ iniconfig==2.0.0
# via
# -r requirements/test.txt
# pytest
isort==5.13.2
# via
# -r requirements/test.txt
# pylint
jaraco-classes==3.4.0
# via keyring
jaraco-context==6.0.1
Expand All @@ -133,10 +121,6 @@ markupsafe==3.0.2
# via
# -r requirements/test.txt
# jinja2
mccabe==0.7.0
# via
# -r requirements/test.txt
# pylint
mdurl==0.1.2
# via markdown-it-py
mongomock==4.2.0.post1
Expand Down Expand Up @@ -169,7 +153,6 @@ pkginfo==1.10.0
platformdirs==4.3.6
# via
# -r requirements/test.txt
# pylint
# tox
# virtualenv
pluggy==1.5.0
Expand All @@ -187,8 +170,6 @@ pygments==2.18.0
# readme-renderer
# rich
# sphinx
pylint==3.2.7
# via -r requirements/test.txt
pymongo==4.10.1
# via -r requirements/test.txt
pyproject-api==1.8.0
Expand Down Expand Up @@ -282,10 +263,6 @@ text-unidecode==1.3
# via
# -r requirements/test.txt
# python-slugify
tomlkit==0.13.2
# via
# -r requirements/test.txt
# pylint
tox==4.23.2
# via -r requirements/test.txt
twine==5.1.1
Expand Down
19 changes: 5 additions & 14 deletions requirements/quality.txt
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,8 @@ asgiref==3.8.1
# -r requirements/test.txt
# django
# django-stubs
astroid==3.2.4
astroid==3.3.5
# via
# -r requirements/test.txt
# pylint
# pylint-celery
cachetools==5.5.0
Expand Down Expand Up @@ -52,9 +51,7 @@ coverage[toml]==7.6.4
# -r requirements/test.txt
# pytest-cov
dill==0.3.9
# via
# -r requirements/test.txt
# pylint
# via pylint
distlib==0.3.9
# via
# -r requirements/test.txt
Expand Down Expand Up @@ -102,7 +99,6 @@ iniconfig==2.0.0
isort==5.13.2
# via
# -r requirements/quality.in
# -r requirements/test.txt
# pylint
jinja2==3.1.4
# via
Expand All @@ -113,9 +109,7 @@ markupsafe==3.0.2
# -r requirements/test.txt
# jinja2
mccabe==0.7.0
# via
# -r requirements/test.txt
# pylint
# via pylint
mongomock==4.2.0.post1
# via -r requirements/test.txt
mypy==1.13.0
Expand Down Expand Up @@ -152,9 +146,8 @@ pycodestyle==2.12.1
# via -r requirements/quality.in
pydocstyle==6.3.0
# via -r requirements/quality.in
pylint==3.2.7
pylint==3.3.1
# via
# -r requirements/test.txt
# edx-lint
# pylint-celery
# pylint-django
Expand Down Expand Up @@ -226,9 +219,7 @@ text-unidecode==1.3
# -r requirements/test.txt
# python-slugify
tomlkit==0.13.2
# via
# -r requirements/test.txt
# pylint
# via pylint
tox==4.23.2
# via -r requirements/test.txt
types-pyyaml==6.0.12.20240917
Expand Down
24 changes: 0 additions & 24 deletions requirements/test.txt
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,6 @@ asgiref==3.8.1
# via
# -r requirements/base.txt
# django
astroid==3.2.4
# via
# -r requirements/base.txt
# pylint
cachetools==5.5.0
# via tox
certifi==2024.8.30
Expand All @@ -33,10 +29,6 @@ colorama==0.4.6
# via tox
coverage[toml]==7.6.4
# via pytest-cov
dill==0.3.9
# via
# -r requirements/base.txt
# pylint
distlib==0.3.9
# via virtualenv
# via
Expand Down Expand Up @@ -65,18 +57,10 @@ idna==3.10
# requests
iniconfig==2.0.0
# via pytest
isort==5.13.2
# via
# -r requirements/base.txt
# pylint
jinja2==3.1.4
# via code-annotations
markupsafe==3.0.2
# via jinja2
mccabe==0.7.0
# via
# -r requirements/base.txt
# pylint
mongomock==4.2.0.post1
# via -r requirements/test.in
mysqlclient==2.2.5
Expand All @@ -93,16 +77,12 @@ pbr==6.1.0
# via stevedore
platformdirs==4.3.6
# via
# -r requirements/base.txt
# pylint
# tox
# virtualenv
pluggy==1.5.0
# via
# pytest
# tox
pylint==3.2.7
# via -r requirements/base.txt
pymongo==4.10.1
# via -r requirements/base.txt
pyproject-api==1.8.0
Expand Down Expand Up @@ -137,10 +117,6 @@ stevedore==5.3.0
# via code-annotations
text-unidecode==1.3
# via python-slugify
tomlkit==0.13.2
# via
# -r requirements/base.txt
# pylint
tox==4.23.2
# via -r requirements/test.in
typing-extensions==4.12.2
Expand Down

0 comments on commit 38183d2

Please sign in to comment.