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

Bump version of django-autotranslate #6632

Merged
merged 1 commit into from
May 12, 2020

Conversation

santteegt
Copy link
Contributor

Description

This PR bumps the version of django-autotranslate to its latest release. It solves the error being raised when building the web docker container using a version of pip>=20.1

Collecting django-autotranslate==1.1.0
  Downloading django-autotranslate-1.1.0.tar.gz (8.7 kB)
    ERROR: Command errored out with exit status 1:
     command: /usr/bin/python3 -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-05q5rnle/django-autotranslate/setup.py'"'"'; __file__='"'"'/tmp/pip-install-05q5rnle/django-autotranslate/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-pip-egg-info-jfqeewod
         cwd: /tmp/pip-install-05q5rnle/django-autotranslate/
    Complete output (7 lines):
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-05q5rnle/django-autotranslate/setup.py", line 25, in <module>
        requirements = [str(ir.req) for ir in install_requirements]
      File "/tmp/pip-install-05q5rnle/django-autotranslate/setup.py", line 25, in <listcomp>
        requirements = [str(ir.req) for ir in install_requirements]
    AttributeError: 'ParsedRequirement' object has no attribute 'req'
    ----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.
ERROR: Service 'web' failed to build: The command '/bin/sh -c pip3 install --upgrade -r test.txt' returned a non-zero code: 1
Refers/Fixes

Refs ankitpopli1891/django-autotranslate#26

@codecov
Copy link

codecov bot commented May 12, 2020

Codecov Report

Merging #6632 into master will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           master    #6632    +/-   ##
========================================
  Coverage   27.23%   27.24%            
========================================
  Files         289      288     -1     
  Lines       27042    26891   -151     
  Branches     4013     3968    -45     
========================================
- Hits         7365     7326    -39     
+ Misses      19397    19297   -100     
+ Partials      280      268    -12     
Impacted Files Coverage Δ
app/retail/views.py 21.79% <0.00%> (-2.71%) ⬇️
app/dashboard/views.py 11.05% <0.00%> (ø)
...rketing/management/commands/no_applicants_email.py 0.00% <0.00%> (ø)
...eting/management/commands/assemble_leaderboards.py 40.05% <0.00%> (ø)
...board/management/commands/pull_codefund_numbers.py
app/dashboard/embed.py 31.60% <0.00%> (+3.44%) ⬆️

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 7021a29...9922ca8. Read the comment docs.

Copy link
Contributor

@octavioamu octavioamu left a comment

Choose a reason for hiding this comment

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

Thanks @santteegt

@gitcoinbot
Copy link
Member

⚡️ A tip worth 0.10000 ETH (19.02 USD @ $190.19/ETH) has been granted to @santteegt for this issue from @owocki. ⚡️

Nice work @santteegt! Your tip has automatically been deposited in the ETH address we have on file.

@thelostone-mc thelostone-mc merged commit e01e800 into gitcoinco:master May 12, 2020
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.

4 participants