Migrate src folder to TypeScript #34
Draft
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Migrates all JavaScript files in
src/to TypeScript while preserving existing behavior and test compatibility.Migration approach
cheminfo-types,ml-spectra-processing, andjcampconverterdependenciesNumberArray/DoubleArrayconversions are needed@default '\t')Files migrated
Kinds.ts,SpectraProcessor.ts,index.tsparseText.ts,parseMatrix.ts,parseJcamp.tsgetMeanData.ts,getNormalizedData.ts,getPostProcessedData.ts, etc.getMetadata.ts,getClasses.ts,getClassLabels.ts,getCategoriesStats.tsgetChart.ts,getBoxPlotChart.ts,getAutocorrelationChart.ts, etc.integration.ts,min.ts,max.ts,minMax.ts,getFromToIndex.tsExample type improvements
Before (JSDoc):
After (TypeScript):
Dependencies added
@types/chroma-js@types/object-hashOriginal prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.