Skip to content

Commit

Permalink
Merge branch 'master' into use-api-tokens-for-jobs
Browse files Browse the repository at this point in the history
  • Loading branch information
pdelboca committed Jul 1, 2022
2 parents 229c613 + 67a5374 commit b421384
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 10 deletions.
1 change: 0 additions & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@ jobs:
- name: Install requirements
run: |
pip install -r requirements.txt
pip install -r pip-requirements.txt
pip install -r dev-requirements.txt
pip install -e .
pip install -U requests[security]
Expand Down
2 changes: 2 additions & 0 deletions ckanext/xloader/loader.py
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,8 @@ def load_csv(csv_filepath, resource_id, mimetype='text/csv', logger=None):
except csv.Error:
logger.warning('Could not determine delimiter from file, use default ","')
delimiter = ','
except UnicodeDecodeError:
raise LoaderError('Could not detect delimiter in this file')

# Setup the converters that run when you iterate over the row_set.
# With pgloader only the headers will be iterated over.
Expand Down
8 changes: 0 additions & 8 deletions pip-requirements.txt

This file was deleted.

4 changes: 3 additions & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
ckantoolkit
messytables==0.15.2
requests[security]>=2.11.1
six>=1.12.0
Unidecode==1.0.22
six>=1.12.0

0 comments on commit b421384

Please sign in to comment.