Skip to content
This repository was archived by the owner on Jun 1, 2025. It is now read-only.

Conversation

@ghiscoding
Copy link
Owner

No description provided.

@bolt-new-by-stackblitz
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@codecov
Copy link

codecov bot commented Mar 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (284a074) to head (51a7e66).
Report is 8 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##            master     #1542    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files            7         8     +1     
  Lines         1031      1625   +594     
  Branches       373       468    +95     
==========================================
+ Hits          1031      1625   +594     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ghiscoding
Copy link
Owner Author

ghiscoding commented Mar 12, 2025

@zewa666 took a look at migrating to Analog following this article and after couple of hours... it's pretty much all done 🚀

One thing that I find is a little negative side, I thought that this would add/replace 1 or 2 deps but it's more like 10 deps (i.e. I'm a little surprised and didn't really want any of the Nx stuff but here it is nonetheless). Apart from that, the rest seems to work, so we'll take it. Ohhh and 1 little thing that the Analog script didn't do by default and broke everything, it created a test-setup.ts (great) but it didn't modify my tsconfig.spec.json to add the file in the include and it threw all king of test bed error but reading the Analog docs and I found it in their docs (perhaps I should open an issue on their project... which I just did)

a quick review might be nice, cheers

@ghiscoding ghiscoding merged commit 7740d38 into master Mar 19, 2025
6 checks passed
@ghiscoding ghiscoding deleted the chore/analog branch March 19, 2025 01:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants