Skip to content

ADD original Satis options skip-errors and no-interaction#2

Merged
kduma merged 1 commit into
kduma-OSS:masterfrom
michaelmeneses:master
Oct 7, 2025
Merged

ADD original Satis options skip-errors and no-interaction#2
kduma merged 1 commit into
kduma-OSS:masterfrom
michaelmeneses:master

Conversation

@michaelmeneses

Copy link
Copy Markdown
Contributor

No description provided.

no-interaction alias no-interaction-s3
@kduma kduma requested a review from Copilot October 6, 2025 21:35

Copilot AI left a comment

Copy link
Copy Markdown

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 adds support for two original Satis command options: skip-errors and no-interaction. The change extends the build state management to track these options and passes them through to the underlying Satis build process.

  • Added skip-errors and no-interaction properties to BuildState class with corresponding getter/setter methods
  • Updated BuildCommand to accept these new CLI options and pass them to the Satis repository build process
  • Modified buildSatisRepository method to conditionally include these options when calling the Satis build command

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
app/Extensions/Internals/BuildStateInterface.php Added interface methods for skip-errors and no-interaction flags
app/Extensions/Internals/BuildState.php Implemented skip-errors and no-interaction properties with getters/setters
app/Commands/BuildCommand.php Added CLI options and integrated them into the Satis build process

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@kduma kduma self-requested a review October 6, 2025 21:36
@kduma kduma merged commit b41953f into kduma-OSS:master Oct 7, 2025
1 of 2 checks passed
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.

3 participants