Skip to content

v0.4.0#9

Merged
michaelahern merged 19 commits intomainfrom
v0.4.0
Jun 20, 2025
Merged

v0.4.0#9
michaelahern merged 19 commits intomainfrom
v0.4.0

Conversation

@michaelahern
Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings June 20, 2025 00:42

This comment was marked as outdated.

@michaelahern michaelahern requested a review from Copilot June 20, 2025 01:39

This comment was marked as outdated.

@michaelahern michaelahern requested a review from Copilot June 20, 2025 02:05
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 updates the service commands for Matterbridge across multiple platforms, adds new install options for frontend port and SSL, and bumps the version to v0.4.0.

  • Modified method signatures in platform, macOS, and Linux implementations to accept extra arguments.
  • Updated sudoers configuration and improved test workflow commands.
  • Adjusted documentation and package version accordingly.

Reviewed Changes

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

Show a summary per file
File Description
src/platform.ts Updated abstract methods to accept args, added user info and mkdir helper
src/mb-service.ts Introduced new install options and updated command handling
src/mac.ts Adjusted install signature and refactored private methods to use shared helpers
src/linux.ts Similar refactoring as macOS; updated sudoers file configuration
package.json Bumped version from 0.3.0 to 0.4.0
README.md Updated option descriptions and usage examples
.github/workflows/test.yml Added chmod on sudoers and tests for updated install options
Comments suppressed due to low confidence (1)

src/linux.ts:103

  • [nitpick] Review if overwriting or creating a new sudoers file might conflict with existing configuration. Consider checking for an existing sudoers entry and appending instead of replacing it if necessary.
        chmodSync(sudoersPath, 0o440);

@michaelahern michaelahern merged commit 92ee158 into main Jun 20, 2025
7 checks passed
@michaelahern michaelahern deleted the v0.4.0 branch June 20, 2025 02:07
@michaelahern michaelahern mentioned this pull request Jun 20, 2025
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