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

Fixing some of the simple tests #4991

Merged
merged 5 commits into from
Mar 24, 2023
Merged

Fixing some of the simple tests #4991

merged 5 commits into from
Mar 24, 2023

Conversation

mojtaba-komeili
Copy link
Contributor

@mojtaba-komeili mojtaba-komeili commented Mar 23, 2023

Patch description
Trying to fix the low hanging test failures.

Before
Screenshot 2023-03-23 at 7 24 12 PM

After
Screenshot 2023-03-23 at 7 23 46 PM

Also added Clean Install.

@mojtaba-komeili
Copy link
Contributor Author

Clean install is now passing too.

@mojtaba-komeili
Copy link
Contributor Author

I know what is wrong with crowdsourcing_tests, it is already fixed in another PR that I am working on.

Copy link
Contributor

@klshuster klshuster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks! you are a life saver

@mojtaba-komeili mojtaba-komeili merged commit bb551e3 into main Mar 24, 2023
@mojtaba-komeili mojtaba-komeili deleted the extraspace branch March 24, 2023 17:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants