Skip to content

Conversation

@haroon7v
Copy link

@haroon7v haroon7v commented Jun 29, 2025

Important

Renamed browser extension in manifest.json and updated submodule URL in .gitmodules, while removing .editorconfig and .pre-commit-config.yaml.

  • Manifest Changes:
    • Renamed extension in manifest.json from "ActivityWatch Web Watcher" to "AssetSonar SaaS Discovery & Usage Monitor".
    • Updated description and version in manifest.json.
  • Repository Configuration:
    • Changed submodule URL in .gitmodules from https://github.com/ActivityWatch/media.git to https://github.com/haroon7v/aw-media.
    • Deleted .editorconfig and .pre-commit-config.yaml files.

This description was created by Ellipsis for a9ff355. You can customize this summary. It will automatically update as commits are pushed.

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to a9ff355 in 21 seconds. Click for details.
  • Reviewed 71 lines of code in 4 files
  • Skipped 0 files when reviewing.
  • Skipped posting 2 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. .gitmodules:3
  • Draft comment:
    Verify updated submodule URL is intentional, as it now points to a fork (haroon7v/aw-media).
  • Reason this comment was not posted:
    Confidence changes required: 50% <= threshold 50% None
2. src/manifest.json:5
  • Draft comment:
    Confirm manifest branding update: new name, description, and version reflect the intended SaaS identity.
  • Reason this comment was not posted:
    Confidence changes required: 50% <= threshold 50% None

Workflow ID: wflow_7RrSFPDUZ4q27322

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

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.

1 participant