Skip to content

xyObjectNormedY#313

Closed
lpatiny wants to merge 14 commits intomainfrom
xyObjectNormedY
Closed

xyObjectNormedY#313
lpatiny wants to merge 14 commits intomainfrom
xyObjectNormedY

Conversation

@lpatiny
Copy link
Member

@lpatiny lpatiny commented Aug 8, 2025

  • feat: add xyObjectNormedY
  • chore: update dependencies and fix eslint
  • chore: update actions to node 24

@lpatiny lpatiny enabled auto-merge (rebase) August 8, 2025 08:41
@codecov
Copy link

codecov bot commented Aug 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.59%. Comparing base (d3b8b15) to head (5f6445d).
⚠️ Report is 8 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #313   +/-   ##
=======================================
  Coverage   96.58%   96.59%           
=======================================
  Files         196      199    +3     
  Lines        5709     5782   +73     
  Branches     1244     1253    +9     
=======================================
+ Hits         5514     5585   +71     
- Misses        195      197    +2     

☔ 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.

@lpatiny
Copy link
Member Author

lpatiny commented Aug 8, 2025

@targos Could you help me with this PR.

The eslint error is related to files that are not in the repo seems to me (matrixTranspose).

I don't know how to solve the TS errors.

@targos
Copy link
Member

targos commented Aug 8, 2025

package.json Outdated
"jscpd": "^4.0.5",
"ml-spectra-fitting": "^4.2.4",
"prettier": "^3.5.3",
"ml-spectra-fitting": "^5.0.1",
Copy link
Member

@targos targos Aug 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't update this package. It is now ESM-only but spectra-processing has not been migrated yet.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok thanks. Indeed my branch is not up-to-date with main ...

@lpatiny lpatiny disabled auto-merge August 8, 2025 09:14
@lpatiny lpatiny closed this Aug 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants