Skip to content

Various enhancements to the build process#65

Open
edwardalee wants to merge 9 commits intomainfrom
build-fixes
Open

Various enhancements to the build process#65
edwardalee wants to merge 9 commits intomainfrom
build-fixes

Conversation

@edwardalee
Copy link
Copy Markdown
Contributor

@edwardalee edwardalee commented Apr 21, 2026

This PR makes the following enhancements to the build process:

  • Fix compiler warnings.
  • Add a lingo update that removes build artifacts, installed dependencies, and lock file, and then rebuilds.
  • Print what is being removed when running lingo clean and lingo update
  • Print what dependency is being loaded before loading it.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR enhances the lingo build tool’s developer feedback and cleanup capabilities, primarily around dependency resolution and cleaning build artifacts.

Changes:

  • Added a cleanall CLI command and documented it in the README.
  • Improved error context for filesystem operations during dependency management and lockfile handling.
  • Added more console output around build/dependency steps and cleanup actions.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
src/util/mod.rs Prints cleanup results when deleting build subdirectories.
src/package/target_properties.rs Fixes serde key for library artifacts deserialization.
src/package/management.rs Adds richer context to FS errors and prints dependency-resolution progress/timing.
src/package/lock.rs Prints lock validation/fetch/checksum timing during init.
src/main.rs Adds cleanall implementation and related deletion helper.
src/backends/mod.rs Prints build-step progress during backend dispatch.
src/args.rs Adds the Cleanall subcommand.
README.md Updates CLI help text to include cleanall.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/main.rs Outdated
Comment thread src/main.rs Outdated
Comment thread src/package/lock.rs Outdated
Comment thread src/util/mod.rs
Comment thread src/package/management.rs Outdated
Comment thread src/backends/mod.rs Outdated
Comment thread src/main.rs Outdated
edwardalee and others added 4 commits April 21, 2026 09:22
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Comment thread src/main.rs Outdated
Comment thread src/package/lock.rs Outdated
Comment thread src/package/lock.rs
Comment thread src/package/management.rs
Comment thread src/package/management.rs
Comment thread src/args.rs Outdated
Comment thread src/main.rs
Copy link
Copy Markdown
Contributor Author

@edwardalee edwardalee left a comment

Choose a reason for hiding this comment

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

Partial responses...

Comment thread src/package/lock.rs Outdated
Comment thread src/package/lock.rs
Comment thread src/package/management.rs
Comment thread src/args.rs Outdated
Comment thread src/main.rs
@edwardalee edwardalee requested a review from tanneberger May 4, 2026 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants