Skip to content

Consolidate requirements.txt file#165

Merged
amercader merged 2 commits intomasterfrom
cleanup-reqs
Jun 30, 2022
Merged

Consolidate requirements.txt file#165
amercader merged 2 commits intomasterfrom
cleanup-reqs

Conversation

@amercader
Copy link
Member

Delete pip-requirements.txt file which included unrelated requirements

@kowh-ai do you know why this file was created when a requirements.txt file already existed?

Delete pip-requirements.txt file which included unrelated requirements
@kowh-ai
Copy link
Collaborator

kowh-ai commented Jun 30, 2022

@amercader - pip-requirements.txt was added in PR to “Update the Travis Setup” #120 - At the time I think I was looking at another CKAN extension as a reference point that had a pip-requirements.txt. I guess all references to pip-requirements.txt should be taken out of the GitHub workflow (test.yml) file also in this PR

@amercader amercader merged commit 67a5374 into master Jun 30, 2022
@amercader
Copy link
Member Author

Thanks @kowh-ai

@amercader amercader deleted the cleanup-reqs branch June 30, 2022 10: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.

2 participants