Stream report #1 from July 2th
TODOs for this stream
- Look at the errors on CI
- Get the tests to be green
- Outline at least 3 use cases for working with a primed profile
What actually happened
I mostly just tore apart the one commit that contained both the added packages and the manual changes from last time.
What was learned
That changing a function definiton from
function x() {}
to
const x = () => {}
can break code because the second one does less implicit capturing than the first one and so code will just subtely break.
In this stream I once again brioke the tests by doing this exact transformation in `javscript-instrument-content-sciope.ts.
Reverting this change is a job for future me.
How is OpenWPM better than it was before
I understand now that just because ESLint complains about something, I can’t just go in there and bow to the suggestions. This was a valuable lesson even if it took two stream to learn.
Outlook for next stream
Since this post is being finished on the 2021-07-12 there is not much use discussing the next stream.