meliorence/react-native-snap-carousel

Setup automated tests and linters on CI

Open

#440 ouverte le 11 déc. 2018

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)JavaScript (2 223 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (10 045 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Is this a bug report, a feature request, or a question?

Choring! 🧹

I like the project and would like to last long and prosper. That's why I propose to setup a CI (e.g. CircleCI or Travis) that would run following jobs:

  • unit/integration tests with Jest – for components and utility functions (feel free to use react-native-testing-library to ease component testing)
  • e2e tests with Detox – these are nice because they allow testing on actual device with fake gestures
  • ESLint or TSLint - eslint/recommended, eslint-config-react-app and eslint-config-prettier are a good start I think, or we can go with tslint
  • type checking – add Flow or TypeScript to the project (Flow is easier to set, but TS gets wider adoption and has lots of 3rd party typings, so you decide)

Unfortunately I likely won't find time to set any of this up, but I wanted to start this initiative anyway, because somebody surely will get some of this done :)

@bd-arc I think the best first step would be for you to set up circle and base on this configuration from react-native-paper, I copy it all the time for new projects :P

Guide contributeur

Direction de recherche
Configurez CI (CircleCI ou Travis) pour exécuter des tests unitaires/d'intégration Jest, des tests e2e Detox, des linters ESLint/TSLint et la vérification de types TypeScript. Utilisez la configuration CI de react native paper comme référence.
Stack technique
javascriptreactreact nativetypescriptnodejs
Domaine
mobiledevopsci cdtesting
Type d'issue
Maintenance
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.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitNode.js
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65