Skip to content

Upgrade all packages with pur #5059

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
merged 5 commits into from
Jan 7, 2019
Merged

Upgrade all packages with pur #5059

merged 5 commits into from
Jan 7, 2019

Conversation

humitos
Copy link
Member

@humitos humitos commented Jan 2, 2019

$ for reqs in `ls requirements`; \
    do pur --skip redis,commonmark,django,docker,celery,gitpython,elasticsearch,pyelasticsearch,mercurial --requirement requirements/$reqs; \
done

$ for reqs in `ls requirements`;
  do pur --skip redis,commonmark,django,docker,celery,gitpython,elasticsearch,pyelasticsearch,mercurial
  --requirement requirements/$reqs;
done
@stsewd
Copy link
Member

stsewd commented Jan 2, 2019

We could upgrade Django to 1.11.17 here too #4756 (comment)

@humitos
Copy link
Member Author

humitos commented Jan 2, 2019

Done!

astroid==1.6.4; python_version < '3'
pylint==2.1.1; python_version > '3'
astroid==2.1.0; python_version > '3'
astroid<1.7 python_version < '3'
Copy link
Member

Choose a reason for hiding this comment

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

A semicolon is missing here

@humitos humitos requested a review from a team January 3, 2019 18:13
@agjohnson
Copy link
Contributor

Perhaps you could add the command you run as an invoke task in our tasks.py? I think the only place it's documented right now is in our PRs upgrading

@humitos
Copy link
Member Author

humitos commented Jan 7, 2019

Perhaps you could add the command you run as an invoke task in our tasks.py?

Good idea! Done at readthedocs/common#27

@humitos humitos merged commit d91807c into master Jan 7, 2019
@humitos humitos deleted the humitos/pur-upgrade-packages branch January 7, 2019 09:45
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.

3 participants