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

Deprecate nbptr_n_2 #690

Merged
merged 2 commits into from
Mar 6, 2017
Merged

Deprecate nbptr_n_2 #690

merged 2 commits into from
Mar 6, 2017

Conversation

benjello
Copy link
Member

No description provided.

CHANGELOG.md Outdated
@@ -1,5 +1,10 @@
# Changelog

## 13.2.1

* Retire la variable `nbptr_n_2`. Elle est inutilisée et obsolète depuis l'introduction des `period`.
Copy link
Member

Choose a reason for hiding this comment

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

Depreciation de variable => changement majeur, sinon @MattiSG ne sera pas content du tout ;)

Copy link
Member

Choose a reason for hiding this comment

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

🍌
C'est pas moi c'est SemVer 😜
C'est-à-dire que si c'est pas moi qui ne suis pas content, ce sont nos utilisateurs qui le seront lorsque leur code ne fonctionnera plus sans qu'ils comprennent pourquoi 😞 On ne peut jamais faire d'hypothèse sur ce qui est utilisé ou non, et je préfère effectivement être intransigeant sur les changements de l'API publique et tenir une promesse.

@fpagnoux
Copy link
Member

fpagnoux commented Mar 6, 2017

Aussi :

  • Nettoyer l'historique de la branche (via un rebase par exemple)
  • Rédiger le Changelog en suivant les nouveaux standards.

@fpagnoux
Copy link
Member

fpagnoux commented Mar 6, 2017

Ok pour le code et le changelog, je vais faire le merge moi-même après #695 pour garder un historique propre et faire des expériences sur l'interface.

@benjello
Copy link
Member Author

benjello commented Mar 6, 2017

Merci @fpagnoux

@fpagnoux fpagnoux force-pushed the deprecate-nbtpr_n_2 branch from 82121f4 to be9f9d8 Compare March 6, 2017 16:25
@fpagnoux
Copy link
Member

fpagnoux commented Mar 6, 2017

Expérience peu concluante, impossible de faire disparaître ce bouton qui pousse au crime.

Rebasé et prêt à être mergé.

@benjello
Copy link
Member Author

benjello commented Mar 6, 2017

Merci @fpagnoux. Je merge dès que les tests seront passés.
Si j'ai bien compris il ne faut jamais faire d'update depuis l'interface github, n'est-ce pas ?

@fpagnoux fpagnoux merged commit 3258282 into master Mar 6, 2017
@fpagnoux fpagnoux deleted the deprecate-nbtpr_n_2 branch March 6, 2017 16:47
@fpagnoux
Copy link
Member

fpagnoux commented Mar 6, 2017

Oui c'est ça, ce bouton produit un "upstream merge" qui complefixie très vite l'historique. La solution est de faire un git rebase, mais il n'y a hélas pas de moyen de le faire via l'UI, il faut passer par la CLI.

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