geolexica / isotc211-glossary

ISO/TC 211 MLGT Online Concepts
2 stars 8 forks source link

Remove or update GHA build #20

Open strogonoff opened 4 years ago

strogonoff commented 4 years ago

There is a left-over GHA workflow that rebuilds concept database YAML from XLSX.

I believe this no longer applies as it is now, since we’re switching to production and an update could overwrite changes made through the app.

(I believe I might have removed this workflow before, but forgot to do it again when I had to re-create my PR after repo went public and fork connection was lost.)

strogonoff commented 4 years ago

@ronaldtse Do you think we can expect occasional changes coming via XLSX during the initial post-Glossarist-transition period?

ronaldtse commented 4 years ago

We won't run the XLSX anymore. Translations that come in via Excel should be manually migrated. Unless we have some "CVS import => new CR" functionality...

skalee commented 4 years ago

Neither I see any reason to keep legacy workflows. That said, it seems reasonable to do some integrity testing in GHA, although I have no idea what these tests could look like. Also, we may consider some brand new release workflow (see https://github.com/geolexica/isotc211.geolexica.org/issues/141).