.pre-commit-config.yaml

@@ -1,15 +1,22 @@
 # @format
 
-# SPDX-FileCopyrightText: 2024 Aminda Suomalainen <suomalainen+git@mikaela.info>
+# SPDX-FileCopyrightText: 2023 Aminda Suomalainen <suomalainen+git@mikaela.info>
 #
 # SPDX-License-Identifier: CC0-1.0
 
 # See https://pre-commit.com for more information
 # See https://pre-commit.ci for more information
 ci:
+  # Forĝejo/Gitea mirrors will autoclose pull requests. This should decrease
+  # the frequency of unnecessary PRs.
+  # https://github.com/pre-commit-ci/issues/issues/83
   autoupdate_schedule: quarterly
   skip: [pnpm-install-dev, prettier, pylint]
 
+# use pypy3 for python
+default_language_version:
+  python: pypy3
+
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
     rev: v4.6.0
@@ -29,49 +36,65 @@ repos:
       - id: check-docstring-first
       - id: check-json
       - id: check-merge-conflict
-      - id: check-toml
+      #- id: check-toml
       #- id: check-xml
       - id: debug-statements
       - id: mixed-line-ending
-        args: [--fix=lf]
+        # .gitattributes is trusted to handle this
+        args: [--fix=no]
       - id: pretty-format-json
         args: [--autofix, --no-ensure-ascii]
-
+        #- id: requirements-txt-fixer
   - repo: https://github.com/pre-commit-ci/pre-commit-ci-config
     rev: v1.6.1
     hooks:
       - id: check-pre-commit-ci-config
-
   - repo: https://github.com/thlorenz/doctoc
     rev: v2.2.0
     hooks:
       - id: doctoc
         # https://github.com/Mikaela/gist/blob/master/doctoc.txt
         args: [--update-only]
-
   - repo: https://github.com/pre-commit/mirrors-mypy
     rev: "v1.10.1"
     hooks:
       - id: mypy
-        additional_dependencies: [types-tabulate]
+        args: [--install-types, --non-interactive]
-
+  - repo: https://github.com/asottile/pyupgrade
-  - repo: https://github.com/pycqa/isort
+    rev: v3.16.0
-    rev: 5.13.2
     hooks:
-      - id: isort
+      - id: pyupgrade
-        name: isort (python)
+        args:
+          [--py310-plus]
+          #  - repo: https://github.com/asottile/reorder-python-imports
+          #    rev: v3.12.0
+          #    hooks:
+          #- id: reorder-python-imports
+          #args: [--py310-plus]
+  - repo: https://github.com/asottile/add-trailing-comma
+    rev: v3.1.0
+    hooks:
+      - id: add-trailing-comma
 
   - repo: https://github.com/psf/black
     rev: 24.4.2
     hooks:
       - id: black
+        #Refer to pyproject.toml
+        #args: ["--line-length", "79", "--target-version", "py310"]
+      #- id: black-jupyter
+      #  - repo: https://github.com/scop/pre-commit-shfmt
+      #    rev: v3.8.0-1
+      #    hooks:
+      #      - id: shfmt
+      #- id: shfmt-docker
 
   - repo: local
     hooks:
       - id: pylint
         name: pylint
-        entry: pylint --exit-zero
+        # Use as many jobs as CPU cores, disable TODO warnings
-        verbose: true
+        entry: pylint -j 0 -d W0511
         language: system
         types_or: [python, pyi]
       - id: pnpm-install-dev
@@ -79,6 +102,7 @@ repos:
         entry: corepack pnpm install -D
         language: system
         always_run: true
+        #verbose: true
         pass_filenames: false
       - id: prettier
         name: prettier

.prettierrc.json

@@ -1,6 +1,7 @@
 {
   "bracketSameLine": true,
   "endOfLine": "auto",
+  "insertPragma": true,
   "plugins": [
     "@prettier/plugin-ruby",
     "prettier-plugin-sh"

.ruby-version

@@ -1 +0,0 @@
-3.3.4

Gemfile

@@ -1,6 +0,0 @@
-source "https://rubygems.org"
-ruby file: ".ruby-version"
-gem "prettier_print"
-gem "syntax_tree"
-gem "syntax_tree-haml"
-gem "syntax_tree-rbs"

Gemfile.lock

@@ -1,40 +0,0 @@
-GEM
-  remote: https://rubygems.org/
-  specs:
-    haml (6.3.0)
-      temple (>= 0.8.2)
-      thor
-      tilt
-    logger (1.6.0)
-    prettier_print (1.2.1)
-    rbs (3.5.2)
-      logger
-    syntax_tree (6.2.0)
-      prettier_print (>= 1.2.0)
-    syntax_tree-haml (4.0.3)
-      haml (>= 5.2)
-      prettier_print (>= 1.2.1)
-      syntax_tree (>= 6.0.0)
-    syntax_tree-rbs (1.0.0)
-      prettier_print
-      rbs
-      syntax_tree (>= 2.0.1)
-    temple (0.10.3)
-    thor (1.3.1)
-    tilt (2.4.0)
-
-PLATFORMS
-  ruby
-  x86_64-linux
-
-DEPENDENCIES
-  prettier_print
-  syntax_tree
-  syntax_tree-haml
-  syntax_tree-rbs
-
-RUBY VERSION
-   ruby 3.3.4p94
-
-BUNDLED WITH
-   2.5.11

pyproject.toml

@@ -1,33 +1,22 @@
-# Black
+# When changing this file, read also .pre-commit-config.yaml
+
 [tool.black]
 line-length = 79
 target-version = ['py310']
-preview = true
 
-# Mypy
 [tool.mypy]
-python_version = '3.10'
+#python-executable = 'pypy3'
-
+python-version = 3.10
-# Pylint
-[tool.pylint.main]
-py-version = "3.10"
-jobs = 0
-
-[tool.pylint.basic]
-include-naming-hint = true
-
-[tool.pylint.format]
-max-line-length = 79
-
-[tool.pylint."messages control"]
-# Disables warnings about TODO
-disable = ["W0511"]
-
-#[tool.pylint.variables]
-#ignored-argument-names = "_.*|^ignored_|^unused_"
 
 # https://black.readthedocs.io/en/stable/guides/using_black_with_other_tools.html
 
-# isort
+# No need to have it uncommented since 79 is the standard.
-[tool.isort]
+#[tool.pylint.format]
-profile = "black"
+#max-line-length = 79
+
+# Not actually used here, see .pre-commit-config.yaml for reorder-python-imports
+# that does the same thing.
+#[tool.isort]
+#profile = "black"
+
+# copy-pasted comments end.

python/keyoxide_gen_hashed_proof.py

@@ -6,9 +6,7 @@ This script generated Keyoxide hashed proofs in argon2id and bcrypt
 PROOF = str(input("Enter the proof: "))
 
 print(
-    """
+    "$argon2id$v=19$m=64,t=512,p=2$mDejbh8SmFFJPjbjf+u9kQ$yQpomARjYKfgEDAZ1jdgnA",
-$argon2id$v=19$m=64,t=512,p=2$mDejbh8SmFFJPjbjf+u9kQ$yQpomARjYKfgEDAZ1jdgnA",
-""",
 )
 
 

python/ssid_check_valid_length.py

@@ -25,8 +25,7 @@ def check_ssid_length(given_ssid_length):
 # Request input or assume "test" on empty.
 GIVENSSID = str(
     input(
-        "Please enter the SSID you are thinking of (preferably 26 chars to \
+        "Please enter the SSID you are thinking of (preferably 26 chars to fit _nomap): ",
-fit _nomap): ",
     )
     or "test",
 )

python/taitotalo/.editorconfig

@@ -0,0 +1,15 @@
+root = true
+
+[*]
+# https://peps.python.org/pep-0008/#indentation
+indent_style = space
+indent_size = 4
+max_line_length = 79
+
+[*.py]
+profile = black
+max_line_length = 79
+#end_of_line = crlf
+
+[*.md]
+end_of_line = lf

python/taitotalo/.gitattributes

@@ -0,0 +1,3 @@
+* text=auto
+# This is for doctoc should I decide to use it in the future
+*.md text eol=lf

python/taitotalo/main.py

@@ -1,7 +1,5 @@
 """
-Tämä on ensimmäinen Taitotalossa kirjoitettu Python koodi koulutuspäivänä
+Tämä on ensimmäinen Taitotalossa kirjoitettu Python koodi koulutuspäivänä yksi. Se vain tulostaa asioita.
-yksi. Se vain tulostaa asioita.
-
 """
 
 # Ensimmäinen kommentti
@@ -10,8 +8,8 @@ print("Tervetuloa")
 # toinen kommentti
 print("esi" + "merkki")
 
-LUKU1 = 5
+luku1 = 5
-LUKU2 = 7
+luku2 = 7
 
-SUMMA = LUKU1 + LUKU2
+summa = luku1 + luku2
-print("Vastaus", SUMMA)
+print("Vastaus", summa)

python/taitotalo/main1.py

@@ -1,12 +1,10 @@
 """
 Sovellus pyytää käyttäjältä nimen, sekä viestin ja tulostaa viestin kolmesti.
-
 """
 
 # Pyydetään käyttäjältä nimi ja viesti, tallennetaan samannimisiin muuttujiin
 nimi = input("Nimi: ")
 viesti = input("Viesti: ")
 
-# Tulostetaan nimi ja kolmesti viesti rivinvaihdolla. Lopuksi poistetaan
+# Tulostetaan nimi ja kolmesti viesti rivinvaihdolla. Lopuksi poistetaan rivinvaihto ja erotin tulostuksesta
-# rivinvaihto ja erotin tulostuksesta
 print(nimi, " sanoo:\n", (viesti + "\n") * 3, end="", sep="")

python/taitotalo/main10.py

@@ -24,13 +24,11 @@ def main():
 
     print("Olen", ikani, "-vuotias", nimeni)
     nimi = input("Mikä sinun nimesi on?: ")
-    # Odotetaan käyttäjän mahdollisesti pyrkivän syöttämään muutakin kuin
+    # Odotetaan käyttäjän mahdollisesti pyrkivän syöttämään muutakin kuin numeroita ja varaudutaan asiaan
-    # numeroita ja varaudutaan asiaan
     try:
         # Tallennetaan kokonaislukuna käyttäjältä pyydetty ikä
         ika = int(input("Entä kuinka vanha olet? "))
-    # Annetaan käyttäjäystävällinen virheilmoitus ja poistutaan
+    # Annetaan käyttäjäystävällinen virheilmoitus ja poistutaan virhekoodilla, jos käyttäjä ei syöttänyt numeroita
-    # virhekoodilla, jos käyttäjä ei syöttänyt numeroita
     except ValueError:
         print("Syötä ikäsi numeroina!")
         sys.exit(1)
@@ -42,8 +40,7 @@ def main():
     # Aloitetaan kokeilemalla onko käyttäjä alle 18.
     if ika < 18:
         print("Olet alaikäinen")
-    # Jos käyttäjä ei ollut alaikäinen, varmistetaan onko hän 18 ja onko hän
+    # Jos käyttäjä ei ollut alaikäinen, varmistetaan onko hän 18 ja onko hän alle 65.
-    # alle 65.
     elif (ika >= 18) and (ika < 65):
         print("Olet täysi-ikäinen")
     # Mikäli ikä on 65 tai yli, käyttäjä on seniori.
@@ -54,7 +51,6 @@ def main():
         print("Jokin meni pieleen. Yritä uudelleen.")
 
 
-# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan
+# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan pääfunktio.
-# pääfunktio.
 if __name__ == "__main__":
     main()

python/taitotalo/main11.py

@@ -29,7 +29,6 @@ def main():
     print()
 
 
-# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan,
+# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan pääfunktio.
-# suoritetaan pääfunktio.
 if __name__ == "__main__":
     main()

python/taitotalo/main12.py

@@ -39,7 +39,6 @@ def main():
             print("Syötä luku!")
 
 
-# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan,
+# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan pääfunktio.
-# suoritetaan pääfunktio.
 if __name__ == "__main__":
     main()

python/taitotalo/main2.py

@@ -6,8 +6,8 @@ import datetime
 
 tanaan = datetime.date.today()
 vuosi = int(tanaan.year)
-NIMI = "Jussi"
+nimi = "Jussi"
 
 print(vuosi)
 print(vuosi == 2014)
-# print(vuosi is vuosi)
+print(vuosi is vuosi)

python/taitotalo/main3.py

@@ -2,12 +2,12 @@
 Sovellus tekee yksinkertaisen salasanatarkistuksen if:illä
 """
 
-KOODISANA = "python"
+koodisana = "python"
 
 print("Tervetuloa")
 syote = input("Anna salasana: ")
 
-if syote == KOODISANA:
+if syote == koodisana:
     print("Matka voi alkaa")
 else:
     print("Yritä uudelleen")

python/taitotalo/main4.py

@@ -1,19 +1,15 @@
 """
-Sovellus kysyy käyttäjän ikää, kertoo alle 18 heidän olevan alaikäinen, 18-65
+Sovellus kysyy käyttäjän ikää, kertoo alle 18 heidän olevan alaikäinen, 18-65 aikuinen ja yli 65 seniori.
-aikuinen ja yli 65 seniori.
-
 """
 
 # Tuo sys -kirjasto, jotta voimme poistua virhekoodiin.
 import sys
 
-# Odotetaan käyttäjän mahdollisesti pyrkivän syöttämään muutakin kuin
+# Odotetaan käyttäjän mahdollisesti pyrkivän syöttämään muutakin kuin numeroita ja varaudutaan asiaan
-# numeroita ja varaudutaan asiaan
 try:
     # Tallennetaan kokonaislukuna käyttäjältä pyydetty ikä
     ika = int(input("Mikä on ikäsi? "))
-# Annetaan käyttäjäystävällinen virheilmoitus ja poistutaan virhekoodilla,
+# Annetaan käyttäjäystävällinen virheilmoitus ja poistutaan virhekoodilla, jos käyttäjä ei syöttänyt numeroita
-# jos käyttäjä ei syöttänyt numeroita
 except ValueError:
     print("Syötä ikäsi numeroina!")
     sys.exit(1)
@@ -21,8 +17,7 @@ except ValueError:
 # Aloitetaan kokeilemalla onko käyttäjä alle 18.
 if ika < 18:
     print("Olet alaikäinen")
-# Jos käyttäjä ei ollut alaikäinen, varmistetaan onko hän 18 ja onko hän
+# Jos käyttäjä ei ollut alaikäinen, varmistetaan onko hän 18 ja onko hän alle 65.
-# alle 65.
 elif (ika >= 18) and (ika < 65):
     print("Olet aikuinen")
 # Mikäli ikä on 65 tai yli, käyttäjä on seniori.

python/taitotalo/main5.py

@@ -1,13 +1,9 @@
-"""
-Tämän skriptin olisi tarkoitus piirtää Suomen lippu turtle-kirjastolla,
-mutta en koskaan tehnyt siitä valmista.
-"""
-
 # Tuodaan turtle -kirjasto
-# Satunnaislukuja
-# import random
 import turtle
 
+# Satunnaislukuja
+import random
+
 # ikkkuna = turtle.Screen()
 
 # Asetetaan turtlen väriksi "meripihka"
@@ -53,37 +49,37 @@ turtle.showturtle()
 
 
 # Piirretään suorakulmio
-VAAKA = 300
+vaaka = 300
-PYSTY = 200
+pysty = 200
 #
-turtle.forward(VAAKA)
+turtle.forward(vaaka)
 turtle.left(90)
-turtle.forward(PYSTY)
+turtle.forward(pysty)
 #
 turtle.left(90)
-turtle.forward(VAAKA)
+turtle.forward(vaaka)
 #
 turtle.left(90)
-turtle.forward(PYSTY)
+turtle.forward(pysty)
 turtle.left(90)
 
 # Piirretään risti
-turtle.forward(VAAKA / 2)
+turtle.forward(vaaka / 2)
 turtle.left(90)
-turtle.forward(PYSTY / 3)
+turtle.forward(pysty / 3)
 
 turtle.right(90)
-turtle.forward(VAAKA / 3)
+turtle.forward(vaaka / 3)
 turtle.left(90)
 
-turtle.forward(VAAKA / 5)
+turtle.forward(vaaka / 5)
 turtle.left(90)
-turtle.forward(VAAKA / 3)
+turtle.forward(vaaka / 3)
 
 turtle.right(90)
-turtle.forward(VAAKA / 3)
+turtle.forward(vaaka / 3)
 
-# turtle.forward(PYSTY / 3)
+# turtle.forward(pysty / 3)
 
 # Pidetään ikkuna auki
 turtle.mainloop()

python/taitotalo/main6.py

@@ -13,25 +13,21 @@ def main():
     Pääfunktio.
     """
 
-    # Pyydetään käyttäjältä kaksi lukua varautuen siihen, ettei käyttäjä
+    # Pyydetään käyttäjältä kaksi lukua varautuen siihen, ettei käyttäjä syötä lukuja oikeassa muodossa.
-    # syötä lukuja oikeassa muodossa.
     try:
         luku1 = float(input("Anna ensimmäinen luku: "))
         luku2 = float(input("Anna toinen luku: "))
-    # Mikäli käyttäjä syöttää jotakin, mitä emme voi muuttaa float:iksi tai
+    # Mikäli käyttäjä syöttää jotakin, mitä emme voi muuttaa float:iksi tai int:iksi, annetaan virheilmoitus ja poistutaan virhekoodiin.
-    # int:iksi, annetaan virheilmoitus ja poistutaan virhekoodiin.
     except ValueError:
         print("Syötä luku!")
         sys.exit(1)
 
-    # Tulostetaan laskutoimitukset. Käytetään int siisteyden vuoksi, vaikka
+    # Tulostetaan laskutoimitukset. Käytetään int siisteyden vuoksi, vaikka uhrataan tuki desimaalien laskemiselle.
-    # uhrataan tuki desimaalien laskemiselle.
     print(int(luku1), "+", int(luku2), "=", int(luku1 + luku2))
     print(int(luku1), "-", int(luku2), "=", int(luku1 - luku2))
     print(int(luku1), "*", int(luku2), "=", int(luku1 * luku2))
 
-    # Varaudutaan siihen, että käyttäjä syöttää luvuksi nollan ja siinä
+    # Varaudutaan siihen, että käyttäjä syöttää luvuksi nollan ja siinä tapauksessa kieltäydytään suorittamasta tehtävää.
-    # tapauksessa kieltäydytään suorittamasta tehtävää.
     if luku1 == 0 or luku2 == 0:
         print("Nollalla ei voi jakaa.")
     else:
@@ -39,7 +35,6 @@ def main():
         print(int(luku1), "/", int(luku2), "=", (luku1 / luku2))
 
 
-# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan
+# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan pääfunktio.
-# pääfunktio.
 if __name__ == "__main__":
     main()

python/taitotalo/main7.py

@@ -33,8 +33,7 @@ def arvaa_numero():
 
         laskuri += 1
 
-        # Pyydetään käyttäjältä luku varautuen virheelliseen syötteeseen. Se
+        # Pyydetään käyttäjältä luku varautuen virheelliseen syötteeseen. Se on tärkeää muuttaa int:iksi tai muuten se on string!
-        # on tärkeää muuttaa int:iksi tai muuten se on string!
         try:
             arvaus = int(input("Arvaa numero? "))
         # Virhesyötteen tapahtuessa
@@ -55,16 +54,13 @@ def arvaa_numero():
             )
             break
 
-        # Muuten avustetaan käyttäjää antamalla vihje. Printistä myös
+        # Muuten avustetaan käyttäjää antamalla vihje. Printistä myös poistetaan erottavat välilyönnit, jotka tehdään itse, jotta arvauksen ja pisteen väliin ei tule välilyöntejä
-        # poistetaan erottavat välilyönnit, jotka tehdään itse, jotta
+        elif arvaus < oikeavastaus:
-        # arvauksen ja pisteen väliin ei tule välilyöntejä
-        if arvaus < oikeavastaus:
             print("Oikea luku on suurempi kuin ", arvaus, ".", sep="")
         else:
             print("Oikea luku on pienempi kuin ", arvaus, ".", sep="")
 
 
-# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan
+# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan pääfunktio.
-# pääfunktio.
 if __name__ == "__main__":
     main()

python/taitotalo/main8.py

@@ -2,12 +2,12 @@
 Sovellus tekee listasta pareja
 """
 
-# Tarvitaan parien muodostamiseen
-import random
-
 # Tuodaan paketti tabulate.
 import tabulate
 
+# Tarvitaan parien muodostamiseen
+import random
+
 
 # Määritetään pääfunktio
 def main():
@@ -23,8 +23,7 @@ def main():
     random_names1 = random.sample(nimet, 3)
     random_names2 = random.sample(nimet2, 3)
 
-    # Luo tuplen, jossa zip yhdistää random_names{1,2} vastaavan kohdan
+    # Luo tuplen, jossa zip yhdistää random_names{1,2} vastaavan kohdan kanssa, jotka sitten laitetaan listaan.
-    # kanssa, jotka sitten laitetaan listaan.
     name_pairs = list(zip(random_names1, random_names2))
 
     print(name_pairs)
@@ -39,7 +38,6 @@ def main():
     print(table)
 
 
-# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan,
+# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan pääfunktio.
-# suoritetaan pääfunktio.
 if __name__ == "__main__":
     main()

python/taitotalo/main9.py

@@ -34,16 +34,13 @@ def main():
     elif tkvalinta == "paperi" and valinta == "sakset":
         print("Voitit, sakset voittavat paperin")
     else:
-        # Tästä tuli vahingossa catch-all virheellisille syötteille, enkä ole
+        # Tästä tuli vahingossa catch-all virheellisille syötteille, enkä ole varma toteutinko kaikki skenaariot
-        # varma toteutinko kaikki skenaariot
         print(
-            "Joko tilannetta ei ole vielä toteutettu tai annoit \
+            "Joko tilannetta ei ole vielä toteutettu tai annoit virheellisen syötteen. Ilmoittaisitko ongelman kehittäjälle?",
-virheellisen syötteen. Ilmoittaisitko ongelman kehittäjälle?"
         )
         print("Tietokone valitsi:", tkvalinta, "syötteesi oli", valinta)
 
 
-# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan
+# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan pääfunktio.
-# pääfunktio.
 if __name__ == "__main__":
     main()

python/taitotalo/requirements.txt

@@ -1,2 +1,2 @@
-# muodosta-parityo-parit.py requires this
+# main8.py requires this
 tabulate

submodules/shell-things

@@ -1 +1 @@
-Subproject commit 73572f12945caee1de71f25cb400ea2764fc62b5
+Subproject commit 074a2a87fb14702ed42615b1246011295ab3b53b