Skip to content

Commit

Permalink
Docker (#34)
Browse files Browse the repository at this point in the history
### To be filled by the PR creator:

* A brief description of the changes made - 

* Do we have clean latest run report attached with this PR?
  * [ ] Yes
  * [ ] No (Please explain why)

* Does the PR contain changes to any BP core file?
  * [ ] Yes (Needs approval from at least 2 people)
  * [ ] No

* Does the PR contain changes to modules shared with other teams?
* [ ] Yes (Needs approval from at least one of the other teams that use
the module)
  * [ ] No

* Is it
  * [ ] New Testcase
  * [ ] Fix


### To be filled by the PR reviewer:

* [ ] Verify the attached run report passed in GitHub Actions (Justify
if local run)

* General
    * [ ] Use the best strategy to locate the elements
    * [ ] Comments wherever the code is not readable by itself
    * [ ] Use of the right data structure for the use case
    * [ ] Reuse logic/functionality as much as possible
    * [ ] Cleanup of any test data that is generated by the tests
    * [ ] No static waits
  • Loading branch information
Tauqir Sarwar authored Mar 18, 2024
2 parents 387b6a8 + 4c71f18 commit 7753726
Showing 1 changed file with 11 additions and 0 deletions.
11 changes: 11 additions & 0 deletions .github/workflows/docker_workflow.yml
Original file line number Diff line number Diff line change
Expand Up @@ -115,6 +115,12 @@ jobs:
--html=report.html \
--self-contained-html \
- name: Docker Container Stop
uses: browserstack/github-actions/setup-local@master
run: |
docker compose --file docker-compose.yml down
if: ${{ always() }}

- name: Upload HTML run report in the Artifacts Folder
uses: actions/upload-artifact@v4
with:
Expand Down Expand Up @@ -214,6 +220,11 @@ jobs:
--tags="$TAGS" --base-url="$BASE_URL" --html=report.html \
--self-contained-html
- name: Docker Container Stop
uses: browserstack/github-actions/setup-local@master
run: |
docker compose --file docker-compose.yml down
if: ${{ always() }}

- name: Upload HTML run report in the Artifacts Folder
uses: actions/upload-artifact@v4
Expand Down

0 comments on commit 7753726

Please sign in to comment.