Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 30, 2026

Summary of the Pull Request

Addresses spacing inconsistencies in IDL attribute declarations that deviated from the file's established conventions.

PR Checklist

  • Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
  • Tests: Added/updated if needed and all pass
  • Localization: All end user facing strings can be localized
  • Dev docs: Added/updated if needed
  • Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Fixed spacing in SaveImage and ExportContainer method declarations in wslaservice.idl to match existing file conventions:

  • Before: [ in, unique ] IProgressCallback * ProgressCallback, [in,out,unique] WSLA_ERROR_INFO * ErrorInfo
  • After: [in, unique] IProgressCallback* ProgressCallback, [in, out, unique] WSLA_ERROR_INFO* ErrorInfo

Changes align with patterns used throughout the file:

  • No extra spaces inside brackets
  • Spaces after commas in multi-part attributes
  • Asterisks attached to type names

Validation Steps Performed

Verified consistency with existing method declarations (PullImage, LoadImage, ImportImage).


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: ptrivedi <1638019+ptrivedi@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP Address feedback on image save and export container implementation Fix IDL attribute spacing inconsistencies in SaveImage and ExportContainer Jan 30, 2026
Copilot AI requested a review from ptrivedi January 30, 2026 20:55
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.

2 participants