-
Notifications
You must be signed in to change notification settings - Fork 1
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
[FIX] Build and test workflow after upstream catchup v4.8.1 #1327
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ear-dev The build is failing at yarn install
. I was facing the same issue while setting up locally.
Updating the git config worked for me git config --global url."https://".insteadOf git://
Reference: Polymer/polymer#1468 (comment)
Thanks @Shailesh351 testing now |
@Shailesh351 we made it past yarn install, now failing on yarn lint. Do you know what 'rocketchat-services' represents? Any ideas about the failure? Seems to be a typescript error. Thanks. |
@ear-dev |
@Shailesh351 looking at the current upstream workflow they have something different in this spot. maybe they fixed a bug that we have not caught up with yet? https://github.com/RocketChat/Rocket.Chat/blob/develop/.github/workflows/build_and_test.yml#L148 |
@ear-dev I checked upstream with version 4.8.1 and also with the latest version. There is no fix. When I ran upstream 4.8.1, it works. So the issue is at our end. I'm looking into it 👍 |
@ear-dev The build failure issue is now fixed. Can you please review it whenever you get a chance? Thanks! |
Thanks @Shailesh351 great work! |
It doesn't include all the fixes for 1329, only what required for the build to work. |
Part of #1329