leecade/react-native-swiper

v1.5.0 code-review request

Open

#243 ouverte le 22 sept. 2016

Voir sur GitHub
 (0 commentaires) (1 réaction) (0 assignés)JavaScript (10 334 stars) (2 346 forks)batch import
help wanted

Description

ref: https://github.com/leecade/react-native-swiper#changelogs

There are two Major changes:

  1. Adjust project structure, publish with src file, use watch-copy script and relative dependencies for improve develop workflow

    "dependencies": {
    "react-native-swiper": "../"
},

  2. ES6 syntex and standard[https://github.com/feross/standard] code style

Please help for more tests and code review, then release 1.5.0, thanks a lot.

@rajkissu @sunnylqm @tasiyo7333 @getnashty

Guide contributeur

Stack technique
javascriptreact
Domaine
mobile
Type d'issue
refactor
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.
half day
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.
clear
Prérequis
React Native basicsES6 syntaxCode review experience
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40
Direction de recherche
Review the project structure changes and ES6 refactoring in the `src` directory. Compare with the previous structure. Check for adherence to standard code style (feross/standard). Look for any missing tests or potential regressions. The maintainer may have additional comments in the linked changelogs.