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.

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

pylint-django plugin issue with wagtail and py38 issue fixed. #97

Merged
merged 2 commits into from May 7, 2020

Conversation

awais786
Copy link
Contributor

@awais786 awais786 commented May 5, 2020

https://openedx.atlassian.net/browse/BOM-1536

  • pylint-django plugin issue with wagtail fixed.
  • run makeupgrade
  • py38 compatibility issue fixed.

@awais786 awais786 closed this May 5, 2020
@awais786 awais786 reopened this May 5, 2020
@awais786 awais786 changed the title upgrade pylint, fixing quality issue on py3.8 pylint-django plugin issue with wagtail fixed. May 6, 2020
@@ -191,15 +192,15 @@ class ProgramPageBranding(Branding):
page = ParentalKey(ProgramPage, on_delete=models.CASCADE, related_name='branding', unique=True)

cover_image = models.ForeignKey(
'wagtailimages.Image',
Image,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

known issue wagtail/wagtail#5538

pylint-django plugin issue with wagtail fixed.
run make upgrade.
@codecov-io
Copy link

codecov-io commented May 7, 2020

Codecov Report

Merging #97 into master will increase coverage by 0.09%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #97      +/-   ##
==========================================
+ Coverage   90.26%   90.35%   +0.09%     
==========================================
  Files          28       28              
  Lines         842      850       +8     
  Branches      110      110              
==========================================
+ Hits          760      768       +8     
  Misses         63       63              
  Partials       19       19              
Impacted Files Coverage Δ
designer/apps/branding/tests/utils.py 100.00% <ø> (ø)
designer/apps/core/signals.py 100.00% <ø> (ø)
designer/apps/core/tests/utils.py 100.00% <ø> (ø)
designer/apps/pages/tests/utils.py 95.23% <ø> (-0.08%) ⬇️
designer/apps/api/v1/serializers.py 89.83% <100.00%> (ø)
designer/apps/api/v1/test_pages_api.py 95.23% <100.00%> (ø)
designer/apps/api/v1/views.py 64.28% <100.00%> (+0.51%) ⬆️
designer/apps/core/apps.py 100.00% <100.00%> (ø)
designer/apps/core/constants.py 100.00% <100.00%> (ø)
designer/apps/core/models.py 100.00% <100.00%> (ø)
... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8791889...02993ef. Read the comment docs.

Switching from nose to pytest.
@awais786
Copy link
Contributor Author

awais786 commented May 7, 2020

@morenol please review.

@awais786 awais786 marked this pull request as ready for review May 7, 2020 11:14
@awais786 awais786 changed the title pylint-django plugin issue with wagtail fixed. pylint-django plugin issue with wagtail and py38 issue fixed. May 7, 2020
@awais786 awais786 merged commit 57c35d7 into master May 7, 2020
@awais786 awais786 deleted the awais786/BOM-1536 branch May 7, 2020 14:25
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.

None yet

6 participants