Skip to content
This repository was archived by the owner on Nov 10, 2023. It is now read-only.

Fix CI on github action desktop build#1159

Merged
mmv08 merged 6 commits intodevelopmentfrom
fix/ci-bypass
Jul 27, 2020
Merged

Fix CI on github action desktop build#1159
mmv08 merged 6 commits intodevelopmentfrom
fix/ci-bypass

Conversation

@matextrem
Copy link
Contributor

No description provided.

@matextrem matextrem added the Bug 🐛 Something isn't working label Jul 24, 2020
@matextrem matextrem self-assigned this Jul 24, 2020
@github-actions
Copy link

github-actions bot commented Jul 24, 2020

ESLint Summary View Full Report

Annotations are provided inline on the Files Changed tab. You can also see all annotations that were generated on the annotations page.

Type Occurrences Fixable
Errors 0 0
Warnings 0 0
Ignored 0 N/A
  • Result: ✅ success
  • Annotations: 0 total

Report generated by eslint-plus-action

@mmv08
Copy link
Contributor

mmv08 commented Jul 24, 2020

@matextrem could you please give an example of warning that didn't let the build happen?

@ghost
Copy link

ghost commented Jul 24, 2020

Travis automatic deployment:
https://pr1159--safereact.review.gnosisdev.com/app

@mmv08
Copy link
Contributor

mmv08 commented Jul 24, 2020

ok I see it in https://github.com/gnosis/safe-react/runs/906180676

this is why we started using react-app-rewired, we have a configuration override which should suppress this warning: https://github.com/gnosis/safe-react/blob/development/config-overrides.js

why doesn't it work?

@matextrem
Copy link
Contributor Author

matextrem commented Jul 24, 2020

The CI fails if it detect warnings. In windows there are 2 warnings related to critical dependencies.

@matextrem matextrem requested a review from fernandomg July 24, 2020 18:54
@ghost
Copy link

ghost commented Jul 24, 2020

Travis automatic deployment:
https://pr1159--safereact.review.gnosisdev.com/app

@mmv08
Copy link
Contributor

mmv08 commented Jul 25, 2020

@matextrem I understand that. We previously had exactly the same issue and fixed it, I pointed you to the file with the solution. If we merge your PR, we will have two solutions for the same problem. Did you investigate why the current solution doesn't work for windows?

Also I'm not really a fan of CI=false, this way we may miss other possible warnings

@ghost
Copy link

ghost commented Jul 27, 2020

Travis automatic deployment:
https://pr1159--safereact.review.gnosisdev.com/app

@mmv08 mmv08 merged commit 8091a9d into development Jul 27, 2020
@mmv08 mmv08 deleted the fix/ci-bypass branch July 27, 2020 22:04
mmv08 pushed a commit that referenced this pull request Jul 28, 2020
* Prevent failed CI if there is any warning

* Update regex

* Eslint error

Co-authored-by: Mati Dastugue <mdastugu@amazon.com>
@mmv08 mmv08 mentioned this pull request Jul 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Bug 🐛 Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants