search

oneapi-src / oneDPL

oneAPI DPC++ Library (oneDPL) https://software.intel.com/content/www/us/en/develop/tools/oneapi/components/dpc-library.html
Apache License 2.0
720 stars 114 forks source link

limited CI #1756

Closed lslusarczyk closed 1 month ago

lslusarczyk commented 1 month ago

temprorarily disabled all tests except DR ones to speed up applying DR review comments

change to be reverted

akukanov commented 1 month ago

@danhoeflinger @dmitriy-sobolev @timmiesmith - is there a "better" way to do what @lslusarczyk wants for CI in the distributed-ranges branch?

danhoeflinger commented 1 month ago

@danhoeflinger @dmitriy-sobolev @timmiesmith - is there a "better" way to do what @lslusarczyk wants for CI in the distributed-ranges branch?

Umm, this is kind of the "quick and dirty" way of doing it, which I don't really object to. It makes it reasonably obvious that it should be removed.

I think there are ways to exclude runs from the matrix based upon branch name matching. That would allow us to do the same with some fewer lines of code I think. Is it necessary in this scenario where the intention is to revert this?
I'm not sure.

https://github.com/orgs/community/discussions/26253

Perhaps there is some better way still that I'm missing...

lslusarczyk commented 1 month ago

Thanks for looking into this. If we find a better way I will revert this change and disable in a new way. For now I'm merging it.

It has an advantage that it is so dirty that nobody will forget about reverting this change.