gip-inclusion / data-inclusion

data·inclusion aggrège les données de l'insertion sociale et professionnelle
https://api.data.inclusion.beta.gouv.fr/api/v0/docs
MIT License
6 stars 1 forks source link

chore: minor tweaks #263

Closed vmttn closed 2 months ago

vmttn commented 3 months ago

pour la config vscode : ok c'est une bonne idée d'inclure la conf vscode, mais alors il faut inclure toute la config vscode (pour le sous-dossier pipeline à minima) + retirer l'exclusion du .gitignore

personnellement j'avais qqchose comme ça (j'ai perdu le fichier en checkout ton commit) :

{
    "[python]": {
        "editor.defaultFormatter": "charliermarsh.ruff"
    },
    "python.autoComplete.extraPaths": [
        "./dags"
    ],
    "python.defaultInterpreterPath": ".venv/bin/python",
    "python.analysis.extraPaths": [
        "./dags"
    ],
    "python.analysis.typeCheckingMode": "basic",
    "python.testing.pytestEnabled": true,
    "python.testing.unittestEnabled": false,
}

je remets ici aussi une extension intéressante qu' @hlecuyer a repérée : https://marketplace.visualstudio.com/items?itemName=innoverio.vscode-dbt-power-user

pour le profile dbt : pas ok, je dév en utilisant dbt via la cli et ce changement casse ça. Je me demande quel problème tu as rencontré ?

vperron commented 2 months ago

OK pour le second commit concernant les variables d'environnement de la targetr DB, on en a discuté, j'ai retiré.

Pour le coup de la config VSCode, j'ai ajouté la tienne à ce que je proposais (qui était juste un premier jet minimal, mais pas de souci pour qu'il soit mieux, au contraire !)

Pour le .gitgnore je suppose que ce que tu voulais dire est que l'on pouvait retirer les règles concernant ce dossier et par conséquent proposer un extensions.json contenant l'extension de @hlecuyer (qui fonctionne très bien et out of the box d'ailleurs, surtout en conservant la config du profiles.yml pour la DB cible ^^ )

Apparemment, VSCode supporte les recommandations d'extensions meme si le .vscode est dans un sous-dossier (ça se voit quand tu ouvres un fichier du dossier en question), j'ai appris un truc ! J'ai donc fait ça, j'espere que ça te va.

Dernière question : est-ce que je migre l'extension dorzey.vscode-sqlfluff dans ce meme sous-dossier puisque a priori on n'aura du SQL que là ?