leecade/react-native-swiper

v1.5.0 code-review request

Open

#243 geöffnet am 22. Sept. 2016

Auf GitHub ansehen
 (0 Kommentare) (1 Reaktion) (0 zugewiesene Personen)JavaScript (10.334 Stars) (2.346 Forks)batch import
help wanted

Beschreibung

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

Contributor Guide

Tech Stack
javascriptreact
Domain
mobile
Issue Type
refactor
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
half day
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
React Native basicsES6 syntaxCode review experience
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40
Research-Richtung
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.