Skip to content
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

chore(workflows):[#828] add new cucumber workflow with integration to… #850

Merged
merged 5 commits into from
Jul 26, 2024

Conversation

ds-jhartmann
Copy link
Contributor

… Xray

Description

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

Copy link

CHANGELOG file was not updated! Make sure to include important changes.


- name: Find previous comment in PR
if: always() && github.event_name == 'pull_request'
uses: peter-evans/find-comment@v3

Check notice

Code scanning / KICS

Unpinned Actions Full Length Commit SHA Note

Action is not pinned to a full length commit SHA.

- name: Create or update comment in PR
if: always() && github.event_name == 'pull_request'
uses: peter-evans/create-or-update-comment@v4

Check notice

Code scanning / KICS

Unpinned Actions Full Length Commit SHA Note

Action is not pinned to a full length commit SHA.
mvn clean verify -P cucumber -Dgroups="$ISSUE_FILTER" -pl irs-cucumber-tests -am --batch-mode 2> irs-cucumber-tests/report-banner.txt

- name: Publish Cucumber Results
uses: EnricoMi/[email protected]

Check notice

Code scanning / KICS

Unpinned Actions Full Length Commit SHA Note

Action is not pinned to a full length commit SHA.
@ds-jhartmann ds-jhartmann marked this pull request as ready for review July 25, 2024 14:42
@ds-jhartmann ds-jhartmann requested review from dsmf and ds-mwesener July 26, 2024 08:32
Copy link
Contributor

@dsmf dsmf left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@ds-mwesener ds-mwesener left a comment

Choose a reason for hiding this comment

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

LGTM

@ds-mwesener ds-mwesener merged commit 88be739 into main Jul 26, 2024
15 checks passed
@ds-mwesener ds-mwesener deleted the chore/828-reactivate-xray-test-integration branch July 26, 2024 09:17
Copy link

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.

3 participants