jamshale
jamshale
I think this is getting pretty close. I'm not opposed to merging it with the unit testing and integration testing coming later. Sonarcloud flagged a few minor things that should...
> It feels odd to me to define a constant for error text. I'm willing to be convinced otherwise, but it feels like it's not really an issue? I'm not...
FYI. I'm pretty sure we can actually merge this with the sonarcloud check failing. So if we think there is adequate test coverage below 80% we should be able to...
Day off in Canada. Will prioritize this for tomorrow morning.
I don't know where the conflicts are coming from. I can't resolve them. If they get resolved we can merge this now.
> @jamshale given that this is just a doc update, I'm guessing the integration test failure is a fluke? Any insights? Yes. This is the one I've seen twice now....
FYI. Sonarcloud is working correctly now. I think it's just reporting stuff that already existed in the changed files. We should still fix or ignore issues if they aren't a...
@dbluhm @ianco Any opinions on this?
Recently merged a new workflow for the integration tests. The usually take just over 40mins and the latest run was a bit faster. They can vary a few minutes depending...
I noticed this but have not dug into it. I'm not sure how this is effecting endorsement but it must be as integration tests are passing on the nightly runs...