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

Run make upgrade-requirements #4937

Merged
merged 1 commit into from Oct 15, 2021
Merged

Conversation

dannyroberts
Copy link
Member

@dannyroberts dannyroberts commented Oct 7, 2021

Had an issue installing from scratch, so I decided to give just upgrading everything a try. We'll see.

ENVIRONMENTS AFFECTED

None

Copy link
Contributor

@millerdev millerdev left a comment

Choose a reason for hiding this comment

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

Looks like the tests passed. Progress!

@@ -3,7 +3,7 @@ CHANGELOG_MD = $(patsubst %.yml, docs/%.md, $(CHANGELOG_YML))
autogen = src/commcare_cloud/help_cache/ansible.txt src/commcare_cloud/help_cache/ansible-playbook.txt docs/commcare-cloud/commands/index.md docs/changelog/index.md src/commcare_cloud/ansible/roles/ebsnvme/files/_vendor/ebsnvme-id $(CHANGELOG_MD)
all : $(autogen)

REQUIREMENTS=$(shell python -c 'import sys; print("requirements%s.txt" % (3 if sys.version_info[0] == 3 else ""))')
REQUIREMENTS=requirements.txt
Copy link
Contributor

Choose a reason for hiding this comment

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

Ha! I can't believe we left this one behind after the Python 3 upgrade. I was looking forward to having it removed. Thanks!

@dannyroberts
Copy link
Member Author

Sweet. I realized today we're removing the VPN which is a "big change", so I'm going to hold onto this for a couple days and then merge it, so that no issues that come up in the next few days need to consider attributing it to this PR versus the VPN change.

@dannyroberts dannyroberts merged commit 0eb74bb into master Oct 15, 2021
@dannyroberts dannyroberts deleted the dmr/run-make-upgrade-requirements branch October 15, 2021 14:39
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

2 participants