README.md: Add note about SPDX license identifiers

I now use this version in my development environment. Eventually I
should add a matrix of versions to use, but I don't know the GitHub
Actions syntax well enough yet.

.drone.yml

@@ -9,6 +9,7 @@ steps:
   commands:
   - id
   - python -V
+  - apt update && apt install -y gcc g++ libicu-dev pkg-config
   - pip install -r requirements-dev.txt
   - pytest
   - python setup.py install
@@ -25,6 +26,7 @@ steps:
   commands:
   - id
   - python -V
+  - apt update && apt install -y gcc g++ libicu-dev pkg-config
   - pip install -r requirements-dev.txt
   - pytest
   - python setup.py install
@@ -41,6 +43,7 @@ steps:
   commands:
   - id
   - python -V
+  - apt update && apt install -y gcc g++ libicu-dev pkg-config
   - pip install -r requirements-dev.txt
   - pytest
   - python setup.py install

.github/workflows/python-app.yml

@@ -16,10 +16,10 @@ jobs:
 
     steps:
     - uses: actions/checkout@v2
-    - name: Set up Python 3.8
+    - name: Set up Python 3.9
       uses: actions/setup-python@v2
       with:
-        python-version: 3.8
+        python-version: 3.9
     - name: Install dependencies
       run: |
         python -m pip install --upgrade pip

CHANGELOG.md

@@ -4,16 +4,19 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## Unreleased
-## Added
+## [0.4.6] - 2021-03-11
+### Added
 - Validation of dcterms.license field against SPDX license identifiers 
 
-## Changed
+### Changed
 - Use DCTERMS fields where possible in `data/test.csv`
 
 ### Updated
 - Run `poetry update` to update project dependencies
 
+### Fixed
+- Output for all fixes should be green, because it is good
+
 ## [0.4.5] - 2021-03-04
 ### Added
 - Check dates in dcterms.issued field as well, not just fields that have the

README.md

@@ -1,7 +1,7 @@
 # DSpace CSV Metadata Quality Checker ![GitHub Actions](https://github.com/ilri/csv-metadata-quality/workflows/Build%20and%20Test/badge.svg) [![Build Status](https://ci.mjanja.ch/api/badges/alanorth/csv-metadata-quality/status.svg)](https://ci.mjanja.ch/alanorth/csv-metadata-quality)
 A simple, but opinionated metadata quality checker and fixer designed to work with CSVs in the DSpace ecosystem (though it could theoretically work on any CSV that uses Dublin Core fields as columns). The implementation is essentially a pipeline of checks and fixes that begins with splitting multi-value fields on the standard DSpace "||" separator, trimming leading/trailing whitespace, and then proceeding to more specialized cases like ISSNs, ISBNs, languages, unnecessary Unicode, AGROVOC terms, etc.
 
-Requires Python 3.7 or greater (3.8 recommended). CSV and Excel support comes from the [Pandas](https://pandas.pydata.org/) library, though your mileage may vary with Excel because this is much less tested.
+Requires Python 3.7 or greater (3.8+ recommended). CSV and Excel support comes from the [Pandas](https://pandas.pydata.org/) library, though your mileage may vary with Excel because this is much less tested.
 
 If you use the DSpace CSV metadata quality checker please cite:
 
@@ -13,6 +13,7 @@ If you use the DSpace CSV metadata quality checker please cite:
 - Validate languages against ISO 639-1 (alpha2) and ISO 639-3 (alpha3)
 - Experimental validation of titles and abstracts against item's Dublin Core language field
 - Validate subjects against the AGROVOC REST API (see the `--agrovoc-fields` option)
+- Validation of licenses against the list of [SPDX license identifiers](https://spdx.org/licenses)
 - Fix leading, trailing, and excessive (ie, more than one) whitespace
 - Fix invalid and unnecessary multi-value separators (`|`) using `--unsafe-fixes`
 - Fix problematic newlines (line feeds) using `--unsafe-fixes`

csv_metadata_quality/fix.py

@@ -77,7 +77,7 @@ def separators(field, field_name):
 
         if match:
             print(
-                f"{Fore.RED}Fixing invalid multi-value separator ({field_name}): {Fore.RESET}{value}"
+                f"{Fore.GREEN}Fixing invalid multi-value separator ({field_name}): {Fore.RESET}{value}"
             )
 
             value = re.sub(pattern, "||", value)

csv_metadata_quality/version.py

@@ -1 +1 @@
-VERSION = "0.4.5"
+VERSION = "0.4.6"

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "csv-metadata-quality"
-version = "0.4.5"
+version = "0.4.6"
 description="A simple, but opinionated CSV quality checking and fixing pipeline for CSVs in the DSpace ecosystem."
 authors = ["Alan Orth <alan.orth@gmail.com>"]
 license="GPL-3.0-only"

setup.py

@@ -14,7 +14,7 @@ install_requires = [
 
 setuptools.setup(
     name="csv-metadata-quality",
-    version="0.4.5",
+    version="0.4.6",
     author="Alan Orth",
     author_email="aorth@mjanja.ch",
     description="A simple, but opinionated CSV quality checking and fixing pipeline for CSVs in the DSpace ecosystem.",

tests/test_check.py

@@ -224,7 +224,7 @@ def test_check_invalid_agrovoc(capsys):
     """Test invalid AGROVOC subject."""
 
     value = "FOREST"
-    field_name = "dc.subject"
+    field_name = "dcterms.subject"
 
     check.agrovoc(value, field_name)
 
@@ -239,7 +239,7 @@ def test_check_valid_agrovoc():
     """Test valid AGROVOC subject."""
 
     value = "FORESTS"
-    field_name = "dc.subject"
+    field_name = "dcterms.subject"
 
     result = check.agrovoc(value, field_name)