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

must be unique for Travis Jobs not supplying a Coveralls Repo Token #13

Closed
talesdsp opened this issue Sep 8, 2020 · 1 comment
Closed

Comments

@talesdsp
Copy link
Owner

talesdsp commented Sep 8, 2020

It happens intermittently, sometimes works, sometimes not without an actual change in any file ( just refreshing the build )

talesdsp added a commit that referenced this issue Sep 8, 2020
Moving coveralls script to `after_success` as probable solution to #13
@talesdsp
Copy link
Owner Author

talesdsp commented Sep 8, 2020

solution found: lemurheavy/coveralls-public#1264 (comment)

talesdsp added a commit that referenced this issue Sep 8, 2020
Remove repeated commands

Closes #13
talesdsp added a commit that referenced this issue Dec 10, 2020
* set base cypress url

* 🔧 (commit) Format commits

Formatting commits allows faster reviews as well as making git history clear

* 🎨 (format) Add lint and format

Linting and formating code, prevents IDE configurations to make 'empty' commits

* 🎨 (imports) Add root imports

With root imports you can just do "~/" and you are in src folder. Ex: "~/assets"
instead of "../../../assets"

* 🎨 (format) Format existing code base

Formatting existing code base according to eslint and prettier's standard

* ✅ (routes) Update tests and snapshots

Removed enzyme and 'fix' tests using real router instead of mocks, resolving
typing errors

* 🏷️ (navigation) Fix returned type

The function `waitButtons()` was returning type `number` instead of
`NodeJS.Timeout`

* 👷 (cypress) Add build config to run cypress

Run e2e tests on Travis

* ✅ (cypress) Add e2e tests

Add e2e tests to ensure app functionality

* 💚 (coveralls) Fix coveralls 422

Moving coveralls script to `after_success` as probable solution to #13

* 🔥 (ci) Remove unused lines

Remove repeated commands

Closes #13

* 🎨 (No scope) refactor code & setup netlify

Improve code readability, and setup netlify redirects for non-root urls

* ✅ (snapshots) Update tests

Fix broken tests
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

No branches or pull requests

1 participant