Skip to content

Add digiroad version to filenames#60

Merged
jannebe merged 1 commit into
mainfrom
irrotus_nro
Feb 10, 2026
Merged

Add digiroad version to filenames#60
jannebe merged 1 commit into
mainfrom
irrotus_nro

Conversation

@jannebe

@jannebe jannebe commented Feb 9, 2026

Copy link
Copy Markdown
Contributor

This change is Reviewable

@jannebe jannebe requested a review from culka February 9, 2026 14:47
@github-actions

github-actions Bot commented Feb 9, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@culka culka left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@culka reviewed 13 files and all commit messages, and made 2 comments.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @jannebe).


import_digiroad_shapefiles.sh line 26 at r1 (raw file):

    echo $DIGIROAD_IRROTUS_NRO > "${DOWNLOAD_TARGET_DIR}/digiroad_${DIGIROAD_IRROTUS_NRO}.txt"
  fi
  rm -f "${DOWNLOAD_TARGET_DIR}/digiroad_irrotus_nro.txt"

Sinänsä turha tehdä poisto, kun alla oleva rivi kuitenkin ylikirjoittaa mitä tiedostossa olisi ollut.


import_digiroad_shapefiles.sh line 27 at r1 (raw file):

  fi
  rm -f "${DOWNLOAD_TARGET_DIR}/digiroad_irrotus_nro.txt"
  echo $DIGIROAD_IRROTUS_NRO > "${DOWNLOAD_TARGET_DIR}/digiroad_irrotus_nro.txt"

Miksi sama numero tallennetaan sekä digiroad_${DIGIROAD_IRROTUS_NRO}.txt hen että digiroad_irrotus_nro.txt:hen?

Ylempää näistä ei taideta lukea missään?

@jannebe

jannebe commented Feb 10, 2026

Copy link
Copy Markdown
Contributor Author

import_digiroad_shapefiles.sh line 26 at r1 (raw file):

Previously, culka (Teemu Mäkinen) wrote…

Sinänsä turha tehdä poisto, kun alla oleva rivi kuitenkin ylikirjoittaa mitä tiedostossa olisi ollut.

">" ei ylikirjoita, vaan lykkää tiedoston perään.

">:" on vähän shell riippuvainen sitten taas.

@jannebe

jannebe commented Feb 10, 2026

Copy link
Copy Markdown
Contributor Author

import_digiroad_shapefiles.sh line 27 at r1 (raw file):

Previously, culka (Teemu Mäkinen) wrote…

Miksi sama numero tallennetaan sekä digiroad_${DIGIROAD_IRROTUS_NRO}.txt hen että digiroad_irrotus_nro.txt:hen?

Ylempää näistä ei taideta lukea missään?

Koska olen laiska ™️ 😄

Eli toinen tietokoneelle, toinen käyttäjille (mulle) että nään heti tiedostolistauksesta mitä on sisässä. Mut voi ottaa pois jos hiertää. 😉

@jannebe jannebe left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jannebe resolved 1 discussion.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @culka).

@culka culka left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@culka made 2 comments and resolved 1 discussion.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @jannebe).


import_digiroad_shapefiles.sh line 26 at r1 (raw file):

Previously, jannebe (Janne Bergman) wrote…

">" ei ylikirjoita, vaan lykkää tiedoston perään.

">:" on vähän shell riippuvainen sitten taas.

Eikö >> lisää perään ja >ylikirjoita. Paitsi jos ympäristön asetukset blokkaa > toiminnan mikäli tiedosto on jo olemassa, jolloin joutuu käyttämään >|.


import_digiroad_shapefiles.sh line 27 at r1 (raw file):

Previously, jannebe (Janne Bergman) wrote…

Koska olen laiska ™️ 😄

Eli toinen tietokoneelle, toinen käyttäjille (mulle) että nään heti tiedostolistauksesta mitä on sisässä. Mut voi ottaa pois jos hiertää. 😉

Tsek. Ei tuosta haittaa ole, Näytti virheelä 🙂

@jannebe jannebe merged commit 04fbc4d into main Feb 10, 2026
5 of 6 checks passed
@jannebe jannebe deleted the irrotus_nro branch February 10, 2026 11:42
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