Skip to content
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

fix ts error in export.ts #1945

Merged

Conversation

ballPointPenguin
Copy link
Contributor

@ballPointPenguin ballPointPenguin commented Feb 27, 2025

Updates to PR #1944
Fixes #1930

Changes

  • fix ts errors in export.ts
  • improved export.ts and pca.ts to better handle group-id mapping
  • typescript syntax and type usage improvements - more clear to understand the pca data structure
  • fix and improve jest export unit tests
  • minor fixes to ensure proper server build

@ballPointPenguin ballPointPenguin merged commit 595d786 into colinmegill/base-clusters-fix Feb 28, 2025
3 checks passed
@ballPointPenguin ballPointPenguin deleted the br/base-cluster-fix-fix branch February 28, 2025 06:36
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.

1 participant