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

Suppression des anciens fichiers de cache de l'auto-complétion #226

Merged
merged 1 commit into from
May 21, 2021

Conversation

Luwangel
Copy link
Collaborator

@Luwangel Luwangel commented May 20, 2021

Cette Pull Request est

  • Un correctif

Description

Reprise de la PR #211 (en plus simple)

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

  • Suppression des anciens fichiers

@Luwangel Luwangel changed the title Suppressiondes anciens fichiers de cache de l'auto-complétion Suppression des anciens fichiers de cache de l'auto-complétion May 20, 2021
Copy link
Collaborator

@Floby Floby left a comment

Choose a reason for hiding this comment

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

Top !
j'ai fait qq test sur l'app deployée https://dev.vitemado.se/226-merge/
et j'ai rien repéré d'anormal.

@Floby
Copy link
Collaborator

Floby commented May 20, 2021

notre expert generation de l'autocomplete @fcamblor c'est ok pour toi ?

@Luwangel Luwangel requested a review from fcamblor May 21, 2021 08:25
@Luwangel Luwangel merged commit b8eb107 into dev May 21, 2021
@Luwangel Luwangel deleted the clean-old-files branch May 21, 2021 14:51
@Luwangel
Copy link
Collaborator Author

J'ai validé (@fcamblor avait validé sur l'ancienne PR). On pourra facilement restaurer les anciens fichiers ça si on découvre un bug.

@fcamblor
Copy link
Collaborator

Yep c'est good (désolé pour la latence :-))

Juste un truc : il reste le __.json qu'on peut supprimer :-) (je vais le faire directement sur dev)

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