Build Related : rename constants.json to constants.js #11292
Conversation
|
|
Running a single test works gulp test --file "test/spec/modules/yieldoneBidAdapter_spec.js" --nolintrunning all tests, it fails. yyyyyyyyyyyyyy .... @dgirardi help plz |
src/constants.js
Outdated
| @@ -0,0 +1,196 @@ | |||
| export default { | |||
There was a problem hiding this comment.
I think this is the same as JSON for the minifier. It wouldn't be able to remove dead definitions; I'm curious and I'd bet it also wouldn't fix the problem you were having.
If it doesn't, what we want is
export const EVENTS = {...};
export const TARGETING_KEYS = {...};
...
or even better would be
export const EVENT_AUCTION_INIT = 'auctionInit';
...
export const TARGETING_KEY_BIDDER = 'hb_bidder';
...
but of course the latter is a lot more work. I'm sorry I didn't make this clearer earlier.
There was a problem hiding this comment.
No worries. The first one I'll definitely do.
The issue currently is that the minifier removes too much 😅
|
I found the issue why |
|
This is such a win for my current research, i think i could have branched from here and go fully on node. |
|
Awesome 🥳 Is there anything I can do to push this? Otherwise updating and resolving merge conflicts becomes cumbersome. And we hit yet another issue with a bidder that doesn't work since the PUC refactoren, which is probably caused by some webpack minification, trimming away actually required constants 🫠 |
ChrisHuie
changed the title
|
Thanks @ChrisHuie |
Type of change
Description of change
Rename
constants.jsontoconstants.jsTODOs left
Other information
constants.json#10829