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] remove usage of reflectivity to get ast methods by default. #6432
[fix] remove usage of reflectivity to get ast methods by default. #6432
Conversation
Build fails: [64] MetacelloNotification: Loaded -> Reflectivity-Tests-tonel.1 --- tonel:///builds/workspace/uest_and_branch_Pipeline_PR-6432/64/src --- cache |
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.
Somehow the build fails, no CI tests are run (I added the printout as a comment)
…, which are a dependency of Reflectivity-Tests
Yes, we are using the CI to test the work, it is not readyl. |
I looked at the code and yes, it makes sense like this |
Issues seem unrelated to me, but I don't remember flaky stuff in Athens, so this is weird. |
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.
OK!
ASTCCache was using reflectivity by default.
We removed it and are using a strategy instead.
This will allow us to work with bootstrap errors.