Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correction de l'utilisation des mauvais fichiers de cache pour l'auto-complétion #211

Closed
wants to merge 4 commits into from

Conversation

Luwangel
Copy link
Collaborator

@Luwangel Luwangel commented May 16, 2021

Les fichiers de cache d'auto-complétion ont été renommés sur la PR #149 car certains ne passaient pas sur Windows.

Le dernier refacto en date a supprimé l'appel aux fichiers renommés : d0dc463#diff-5f7f40c0d5c6024f626185600d5905793cd993fb722a6159875052c956078653.

Todo

  • Ajout d'une option pré-fixant le nom des fichiers d'auto-complétion
  • Correction des tests
  • Suppression des anciens fichiers

@fcamblor
Copy link
Collaborator

fcamblor commented May 17, 2021

@Luwangel il y a un truc que je ne m'explique pas... je vois bien la régression introduite sur d0dc463, mais pourtant, lorsque je regarde la prod VMD, je vois des autocompletes qui partent sur des vmd_*.json alors que la MEP d'hier soir aurait dû embarquer cette régression, non ?

Pour moi, il est important de se poser la question afin de savoir si on donne un petit sursis aux "vieux" fichiers d'autocomplete, dans le cas où ces derniers auraient pu avoir été utilisé/mis en cache par une vieille version en prod.

Dans le cas contraire, je suis complètement d'avis de m'en séparer pour éviter les erreurs futures :-)

@Luwangel
Copy link
Collaborator Author

Luwangel commented May 17, 2021

@fcamblor Et bien figure toi que je viens de comprendre pourquoi. J'ai regardé les commits d'hier soir, et @Floby a fait une modif sur master :D 4dc698f

@fcamblor
Copy link
Collaborator

ok cool, ben let's go alors (le temps qu'on MEP, les gens auront switché sur la nouvelle version en vmd_*.json

(je laisse la PR encore un peu ouverte au cas où d'autres contributeurs souhaitent redire quelque chose)

@Floby
Copy link
Collaborator

Floby commented May 20, 2021

yup j'ai pris la modif au passage la dernière fois :)
je ferme

@Floby Floby closed this May 20, 2021
@Luwangel
Copy link
Collaborator Author

Luwangel commented May 20, 2021

@Floby La PR supprimait surtout les vieux fichiers :( Tu ne veux pas qu'on la garde ?

Mise à jour : on ne garde que la suppression des anciens fichiers

@Luwangel Luwangel deleted the fix-autocomplete-cache branch May 20, 2021 14:48
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.

3 participants