Skip to content

feat: ucc-gen validate command support #1771

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

hetangmodi-crest
Copy link
Contributor

Issue number:
ADDON-76818

PR Type

What kind of change does this PR introduce?

  • Feature
  • Bug Fix
  • Refactoring (no functional or API changes)
  • Documentation Update
  • Maintenance (dependency updates, CI, etc.)

Summary

Changes

Introduced a new command ucc-gen validate, which will validate the user provided add-on using splunk-appinspect. For now, we have only included cloud tag while validating add-ons.

User experience

Users will now be able to validate their apps and add-ons during development time to ensure that their add-on meets
Splunk's standards and best practices.

ucc-gen validate --addon-path <path_for_addon>

Checklist

If an item doesn't apply to your changes, leave it unchecked.

Review

  • self-review - I have performed a self-review of this change according to the development guidelines
  • Changes are documented. The documentation is understandable, examples work (more info)
  • PR title and description follows the contributing principles
  • meeting - I have scheduled a meeting or recorded a demo to explain these changes (if there is a video, put a link below and in the ticket)

Tests

See the testing doc.

  • Unit - tests have been added/modified to cover the changes
  • Smoke - tests have been added/modified to cover the changes
  • UI - tests have been added/modified to cover the changes
  • coverage - I have checked the code coverage of my changes (see more)

Demo/meeting:

Reviewers are encouraged to request meetings or demos if any part of the change is unclear

@hetangmodi-crest hetangmodi-crest self-assigned this May 29, 2025
@hetangmodi-crest hetangmodi-crest added the enhancement New feature or request label May 29, 2025
@hetangmodi-crest hetangmodi-crest marked this pull request as ready for review May 30, 2025 11:21
@hetangmodi-crest hetangmodi-crest requested review from a team as code owners May 30, 2025 11:21
Copy link

Code Coverage 🎉

Type PR Develop Change Status
Line Coverage 88.28% 88.26% 0.02% 🟢 Increased
Branch Coverage 81.89% 81.93% 0.05% 🔴 Decreased

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant