Remove ci warnings about linking of openssl brew

[mcbarton]

Currently we have a warning in the ci about files being overwritten during the brew link stage for openssl. By removing openssl we should stop this warning appearing when the ci runs.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.57%. Comparing base (29588be) to head (143a494).
Report is 13 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #339      +/-   ##
==========================================
+ Coverage   74.43%   74.57%   +0.14%     
==========================================
  Files           8        8              
  Lines        3204     3214      +10     
==========================================
+ Hits         2385     2397      +12     
+ Misses        819      817       -2     

see 3 files with indirect coverage changes

see 3 files with indirect coverage changes

[vgvassilev]

LGTM! Feel free to merge when the bots are happy.

[mcbarton]

@vgvassilev I eventually found a solution to stop the warning about linking openssl when upgrading, but its kind of a fudge. I just redirect the standard output and error when upgrading openssl, so that the Github ci doesn't detect the warning. Is this solution ok to merge?

[vgvassilev]

@vgvassilev I eventually found a solution to stop the warning about linking openssl when upgrading, but its kind of a fudge. I just redirect the standard output and error when upgrading openssl, so that the Github ci doesn't detect the warning. Is this solution ok to merge?

Yeah, we do what we can ;)

[vgvassilev]

LGTM!