tests: Remove some obsolete reasons to skip tests#1145
Conversation
The issues are either fixed or the distribution is no longer tested in our CI.
Summary of ChangesHello @vojtechtrefny, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request streamlines the test suite by eliminating outdated test skip configurations. By removing these no longer relevant skips, the CI system will now execute a more comprehensive set of tests, ensuring better coverage and catching potential regressions on platforms where previous issues have been resolved or are no longer part of the active testing matrix. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request removes several obsolete test skips from the tests/skip.yml configuration file. The removals are justified by the pull request description, which states that the underlying issues have been fixed or the corresponding distributions are no longer part of the CI test matrix. The changes are straightforward and correct. This cleanup helps keep the test configuration up-to-date and ensures that tests are run on platforms where they are now expected to pass. The changes look good and I have no further comments.
5294d3c
into
storaged-project:master
The issues are either fixed or the distribution is no longer tested in our CI.