Skip to content

Conversation

hetangmodi-crest
Copy link
Contributor

Issue number: -

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

Added a smoke test to verify the UI files to be present in the .whl file of UCC.

User experience

No change, just adding a smoke test to verify all the UI files that should be present.

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 Oct 3, 2025
@hetangmodi-crest hetangmodi-crest marked this pull request as ready for review October 6, 2025 12:57
@hetangmodi-crest hetangmodi-crest requested a review from a team as a code owner October 6, 2025 12:57
kkedziak-splunk
kkedziak-splunk previously approved these changes Oct 6, 2025
@hetangmodi-crest hetangmodi-crest merged commit 8bf7e65 into develop Oct 13, 2025
81 checks passed
@hetangmodi-crest hetangmodi-crest deleted the test/add-test-to-check-UI-files branch October 13, 2025 08:32
@github-actions github-actions bot locked and limited conversation to collaborators Oct 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants