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

[FIX] Build and test workflow after upstream catchup v4.8.1 #1327

Merged
merged 9 commits into from
Oct 25, 2022

Conversation

ear-dev
Copy link

@ear-dev ear-dev commented Oct 18, 2022

Part of #1329

@ear-dev ear-dev changed the title fix workflow [FIX] fix workflow Oct 18, 2022
Copy link

@Shailesh351 Shailesh351 left a 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)

@ear-dev
Copy link
Author

ear-dev commented Oct 20, 2022

@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

@ear-dev
Copy link
Author

ear-dev commented Oct 20, 2022

@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.

@Shailesh351
Copy link

@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 rocketchat-services is one of the RC packages part of the mono repo. The error we are getting is due to the lint issue. Somehow it's scanning for lint issues inside the node_modules folder, which it should not.

@ear-dev
Copy link
Author

ear-dev commented Oct 20, 2022

@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

@Shailesh351
Copy link

@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 👍

@Shailesh351 Shailesh351 changed the title [FIX] fix workflow [FIX] Build and test workflow after upstream catchup v4.8.1 Oct 21, 2022
@Shailesh351
Copy link

@ear-dev The build failure issue is now fixed. Can you please review it whenever you get a chance? Thanks!

@Shailesh351 Shailesh351 self-assigned this Oct 25, 2022
@ear-dev
Copy link
Author

ear-dev commented Oct 25, 2022

Thanks @Shailesh351 great work!
Does this include all the fixes from #1329 as well? It looks like it includes some of them at least?

@Shailesh351
Copy link

It doesn't include all the fixes for 1329, only what required for the build to work.

@ear-dev ear-dev merged commit eb2f8e5 into develop Oct 25, 2022
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

Successfully merging this pull request may close these issues.

3 participants