You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "aremily (via GitHub)" <gi...@apache.org> on 2023/10/27 01:17:50 UTC

[PR] Import Mode fixes build. [commons-crypto]

aremily opened a new pull request, #260:
URL: https://github.com/apache/commons-crypto/pull/260

   The Mode import was removed in a previous commit, which broke the build.  This PR reapplies the required imports.


-- 
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@commons.apache.org

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


Re: [PR] Import Mode fixes build. [commons-crypto]

Posted by "sebbASF (via GitHub)" <gi...@apache.org>.
sebbASF merged PR #260:
URL: https://github.com/apache/commons-crypto/pull/260


-- 
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@commons.apache.org

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


Re: [PR] Import Mode fixes build. [commons-crypto]

Posted by "sebbASF (via GitHub)" <gi...@apache.org>.
sebbASF commented on PR #260:
URL: https://github.com/apache/commons-crypto/pull/260#issuecomment-1782502720

   I don't see evidence of a build failing because of a missing import?


-- 
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@commons.apache.org

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


Re: [PR] Import Mode fixes build. [commons-crypto]

Posted by "aremily (via GitHub)" <gi...@apache.org>.
aremily commented on PR #260:
URL: https://github.com/apache/commons-crypto/pull/260#issuecomment-1782145845

   [buildlog.txt](https://github.com/apache/commons-crypto/files/13184093/buildlog.txt)
   


-- 
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@commons.apache.org

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


Re: [PR] Import Mode fixes build. [commons-crypto]

Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #260:
URL: https://github.com/apache/commons-crypto/pull/260#issuecomment-1782495203

   ## [Codecov](https://app.codecov.io/gh/apache/commons-crypto/pull/260?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
   > Merging [#260](https://app.codecov.io/gh/apache/commons-crypto/pull/260?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) (83302b8) into [master](https://app.codecov.io/gh/apache/commons-crypto/commit/5dd8e29fc0495d44a464bf2eede00431a3e6597e?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) (5dd8e29) will **not change** coverage.
   > The diff coverage is `n/a`.
   
   ```diff
   @@            Coverage Diff            @@
   ##             master     #260   +/-   ##
   =========================================
     Coverage     74.82%   74.82%           
     Complexity      439      439           
   =========================================
     Files            38       38           
     Lines          1819     1819           
     Branches        178      178           
   =========================================
     Hits           1361     1361           
     Misses          348      348           
     Partials        110      110           
   ```
   
   
   
   :mega: We’re building smart automated test selection to slash your CI/CD build times. [Learn more](https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
   


-- 
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: notifications-unsubscribe@commons.apache.org

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


Re: [PR] Import Mode fixes build. [commons-crypto]

Posted by "sebbASF (via GitHub)" <gi...@apache.org>.
sebbASF commented on PR #260:
URL: https://github.com/apache/commons-crypto/pull/260#issuecomment-1782698390

   I see now - these classes are not built by default


-- 
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@commons.apache.org

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