-
Notifications
You must be signed in to change notification settings - Fork 0
Added support for showing cad report #3
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this 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 introduces support for displaying BrowserStack CAD reports by adding new scripts for API communication and configuration setup, along with corresponding CircleCI command and example YAML files.
- Implements CAD report fetching and polling in a new shell script (show_report.sh).
- Adds a configuration script (set_browserstack_config.sh) to set BrowserStack credentials and build names.
- Provides updated CircleCI command definitions and example configurations to integrate the new functionality.
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
File | Description |
---|---|
src/scripts/show_report.sh | New script to make API calls, poll for report status, and generate HTML/text reports. |
src/scripts/set_browserstack_config.sh | New script to fetch CircleCI workflow details and set BrowserStack environment variables. |
src/examples/show_report.yml | Example demonstrating how to integrate the CAD report functionality into CircleCI. |
src/commands/show_report.yml | Command configuration to run the report script and archive the generated report. |
src/commands/set_browserstack_config.yml | Command configuration to run the BrowserStack configuration script. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we change the orb name instead of show reports to something like browserstack test reports , can take product input if needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.