-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Do a bundler run as part of the CI #1418
Comments
Not sure if it fits in the "bundler" category, but there are cases where bundlers, such as For example, the use case that I have is being able to use |
@mfw78 good point. I quickly checked and it seems that |
As for the description just putting it in my words. I think before making it a bounty let's agree on boundaries of what is bare coverage. The thing is if we have such a pipeline step configured then we'll need to support it in the same fashion as |
So long as this meets the requirements of ensuring: "I'm a developer building in XYZ framework and @waku/sdk just works" where XYZ is in the list of the aforementioned frameworks. I'm no bundler expert and have wasted many hours digging into the dependencies abyss. Making this accessible, and preserving it's accessibility is key. |
Well that's the thing. If we do not want an explosion of pipelines then we can just add pipelines for web frameworks but not bundlers. |
Then let me summarise:
To me, it seems those bullet points are very similar to waku-org/bounties#2 and only if we will keep those light weight (like a button) and quickly run in pipelines (per release e.g) then this bounty would make sense. An additional step could be a combination of #52 and those simple web apps with |
@weboko to create a react example, and the other framework examples can be setup as bounties. |
Todo:
|
An example, right now, using |
What's the error? did you open an issue? @helpisdev also mentioned an issue with latest version of |
Sorry for the delay in getting back to this one. Currently in the environment:
Using A repository for reproducing this error is available at https://github.com/rndlabs/pastebee. |
hey @mfw78, strangely I have the app working with Maybe worth trying if you didn't:
|
This is a feature request
Problem
js-waku depends on novel packages such as js-libp2p family of libraries but also cryptography related libraries.
These dependencies change fast and do have breaking changes.
While these library target both browser and NodeJS environment, the biggest users are NodeJS (Lodestar Eth node).
Hence, regressions that only impact the browser can and have been introduced in the past. The most impactful one being the introduction of a NodeJS API usage, that then requires a polyfill in the browser or react native environment.
Polyfills mean added configuration to the bundler for js-waku maintainers and users, it also increases the final size of the web app using js-waku.
Proposed Solutions
Run as part of the CI/tests, a bundling of the library with popular bundlers. The default config of the bundlers should be used to ensure minimum friction of js-waku users.
List of bundlers to consider (some items in this list might not be bundlers):
Notes
Would be good to review best practice for TypeScript library. In my time-boxed search, I could not find anything promising. Yet, I believe I have seen this pattern in the past.
Most likely, we want to have some very simple applications using
@waku/sdk
as a test.The text was updated successfully, but these errors were encountered: