search

ncasuk / amf-check-writer

Library to write AMF compliance checks
BSD 3-Clause "New" or "Revised" License
0 stars 4 forks source link

Cannot use relative path for spreadsheets dir in create-cvs #1

Closed joesingo closed 3 years ago

joesingo commented 6 years ago

Something is going wrong when using a relative path for spreadsheets dir with create-cvs:

(venv) vagrant@localhost:/vagrant/AMF_CVs$ create-cvs spreadsheets ./AMF_CVs
WARNING: Expected to find file at 'spreadsheets/spreadsheets/Common.xlsx/Variables - Air.tsv'
WARNING: Expected to find file at 'spreadsheets/spreadsheets/Common.xlsx/Variables - Land.tsv'
WARNING: Expected to find file at 'spreadsheets/spreadsheets/Common.xlsx/Variables - Sea.tsv'
WARNING: Expected to find file at 'spreadsheets/spreadsheets/Common.xlsx/Dimensions - Air.tsv'
WARNING: Expected to find file at 'spreadsheets/spreadsheets/Common.xlsx/Dimensions - Land.tsv'
WARNING: Expected to find file at 'spreadsheets/spreadsheets/Common.xlsx/Dimensions - Sea.tsv'

It is looking under spreadsheets/spreadsheets instead of just spreadsheets. Using absolute path works as expected.

This may also affect create-yaml-checks.

agstephens commented 3 years ago

@gapintheclouds: please can you check whether this is still a problem. I suspect it is outdated and can be closed.

agstephens commented 3 years ago

Outdated.