Transpilation Optimization

HaidarJbeily7 commented 2 weeks ago

Overview:

Added utility function to skip unnecessary transpilation when source files haven't been modified. This optimization compares file modification timestamps to determine if retranspilation is needed, improving build performance.

Changes:

Added unit tests to verify skip/retranspile behavior
Implemented timestamp comparison logic
Handled case when transpiled file doesn't exist

PR-Codex overview

This PR refactors the transpilation process by introducing a prepare function, modifying the transpile function to use it, and adding a needsRetranspile check to optimize file processing. It also enhances test cases for retranspilation based on file modifications.

Detailed summary

Renamed transpile function to prepare.
Added a new transpile function that calls prepare.
Introduced retranspile function to handle cases when files are modified.
Implemented needsRetranspile function to check modification times.
Updated transform function to use needsRetranspile.
Enhanced test cases for verifying retranspilation behavior.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

HaidarJbeily7 commented 1 week ago

@maxonfjvipon I have made the requested changes.

HaidarJbeily7 commented 1 week ago

Closes: #37

maxonfjvipon commented 1 week ago

@rultor merge

rultor commented 1 week ago

@rultor merge

@maxonfjvipon OK, I'll try to merge now. You can check the progress of the merge here.

rultor commented 1 week ago

@rultor merge

@maxonfjvipon Done! FYI, the full log is here (took me 20min).

maxonfjvipon commented 1 week ago

@HaidarJbeily7 Thanks

objectionary / eo2js