shouldjs/should.js

Is it time to deprecate Object.prototype extension?

Open

#146 ouverte le 20 juin 2017

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)JavaScript (1 906 stars) (131 forks)batch import
discussionhelp wanted

Description

There's more and more changes like this: https://github.com/nodejs/node/pull/6055 where objects no longer inherit from Object.prototype

Is it time to deprecate this feature in favor of a global variable or something like that?

Feel free to close this if you're tired of discussing Object.prototype, I understand.

Node.js version?

6

Should.js version?

any

How to reproduce it?

try asserting on the result of querystring.parse()

Guide contributeur

Stack technique
javascriptnodejs
Domaine
testing
Type d'issue
research
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
JavaScript basicsNode.js ecosystemObject.prototype knowledge
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30
Direction de recherche
Investigate the linked PR (nodejs/node#6055) to understand the change where objects no longer inherit from Object.prototype. Review the should.js codebase for current Object.prototype extension and discuss whether deprecation is appropriate. Consider alternatives like a global variable.