Sheridan C Rawlins

Results 97 comments of Sheridan C Rawlins

@SyntaxNode @linux019 - please take another look - the fiftyonedegrees module is excluded in its entirety now (though, there still needs to be cgo and !cgo files in the modules...

> > There is an issue, this module requires CGO, but PBS traverses all modules files subtree and and adds all modules to autoload. > > Agreed @linux019. We'll need...

**DONE** > I think you can add a simple unit test. It shouldn't allow to build the image without cgo > > ``` > func TestModuleBuilder(t *testing.T) { > _,...

> My understanding is that this is not about the PBS Adapter timeout at all. **AGREE**: https://github.com/prebid/prebid-server/blob/master/config/config.go#L1336-L1353 As I understand it, globally at the moment, there are a few settings...

In an application that does quite a bit of json munging reshaping, the prevailing solution is to use a combination of gjson and sjson. While I'm aware of the cloudwego.gjson...

Similar to what I was saying about #731, given the starting point of ```json { "foo": { "w": 123, "h": 345 } } ``` You may wish to move from...

Another aspect that may be a flag or something - not sure how often this would play in but just saying… there might be a desire to prune empty… ```json...

Yeah, that's not really desired - I guess, if ast is completely disjoint from the rest of the parsing and Pretouch stuff, maybe this isn't easy, but was hoping to...

Actually… I think we didn't actually try with 2.70.0 - there were two cases where the downgrade was done and the place we were eyeballing didn't have the downgrade… we're...

v2.70.0 actually _DOES_ work afterall, still. ``` 00:01:02 + gh pr create --label autoupdate-deals --title '[autoupdate][deals][20250428-232318] Update files from source folder' --body 'This PR is a result of script, which...