You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "liurenjie1024 (via GitHub)" <gi...@apache.org> on 2024/07/15 13:16:02 UTC

[I] Use cargo-nextest to run tests. [iceberg-rust]

liurenjie1024 opened a new issue, #459:
URL: https://github.com/apache/iceberg-rust/issues/459

   [cargo-nextest](https://nexte.st/) provides better interface and faster execution speed to rust tests, we should move to it to improve developer experience.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [I] Use cargo-nextest to run tests. [iceberg-rust]

Posted by "vivek378521 (via GitHub)" <gi...@apache.org>.
vivek378521 commented on issue #459:
URL: https://github.com/apache/iceberg-rust/issues/459#issuecomment-2230069011

   Should this change be made on the CI, too? or just makefile?
   
   I see in the nextest docs that docs-test isn't supported, the rest can be ported. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [I] Use cargo-nextest to run tests. [iceberg-rust]

Posted by "vivek378521 (via GitHub)" <gi...@apache.org>.
vivek378521 commented on issue #459:
URL: https://github.com/apache/iceberg-rust/issues/459#issuecomment-2230189772

   I have raised a PR for this change: https://github.com/apache/iceberg-rust/pull/464
   
   Please review.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [I] Use cargo-nextest to run tests. [iceberg-rust]

Posted by "liurenjie1024 (via GitHub)" <gi...@apache.org>.
liurenjie1024 commented on issue #459:
URL: https://github.com/apache/iceberg-rust/issues/459#issuecomment-2230095607

   > Should this change be made on the CI, too? or just makefile?
   > 
   > I see in the nextest docs that docs-test isn't supported, the rest can be ported.
   
   It's better to change them in both places.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [I] Use cargo-nextest to run tests. [iceberg-rust]

Posted by "Fokko (via GitHub)" <gi...@apache.org>.
Fokko closed issue #459: Use cargo-nextest to run tests.
URL: https://github.com/apache/iceberg-rust/issues/459


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [I] Use cargo-nextest to run tests. [iceberg-rust]

Posted by "Fokko (via GitHub)" <gi...@apache.org>.
Fokko commented on issue #459:
URL: https://github.com/apache/iceberg-rust/issues/459#issuecomment-2503870028

   Seems like `nextest` has some limitations, see https://github.com/apache/iceberg-rust/pull/464. I'll close this issue for now.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org