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.

Sign up for GitHub

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

Jump to bottom

add reporting after screenshot tests #33

Merged
merged 10 commits into from
May 3, 2022
Merged

add reporting after screenshot tests #33

merged 10 commits into from
May 3, 2022

Conversation

valpinkman
Copy link
Member

New

  • added a new action in tools to generate the comment to add after PR

Updated

  • workflow desktop_test.yml (now desktop-test.yml) with a new reporting step after the screenshots have been taken

@vercel
Copy link

vercel bot commented May 2, 2022 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
live-common-tools ✅ Ready (Inspect) Visit Preview May 3, 2022 at 3:18PM (UTC)
2 Ignored Deployments
Name Status Preview Updated
native-ui-storybook ⬜️ Ignored (Inspect) May 3, 2022 at 3:18PM (UTC)
react-ui-storybook ⬜️ Ignored (Inspect) May 3, 2022 at 3:18PM (UTC)

@changeset-bot
Copy link

changeset-bot bot commented May 2, 2022 •
edited
Loading

⚠️ No Changeset found

Latest commit: 44a9f4a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions github-actions bot added automation CI/CD stuff tools Has changes in tools labels May 2, 2022
elbywan
elbywan approved these changes May 3, 2022
View reviewed changes
Copy link
Contributor

@elbywan elbywan left a comment

Choose a reason for hiding this comment

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

👌 👍

@github-actions
Copy link

github-actions bot commented May 3, 2022 •
edited by valpinkman
Loading

@valpinkman

Screenshots: ❌

It seems this PR contains screenshots that are different from the base branch.
If you are sure all those changes are correct, you can comment on this PR with /generate-screenshots to update those screenshots.

Make sure all the changes are correct before running the command, as it will commit and push the new result to the PR.

linux

Actual Diff Expected
help-drawer-actual help-drawer-diff help-drawer-expected
help-drawer-actual help-drawer-diff help-drawer-expected
market-btc-page-actual market-btc-page-diff market-btc-page-expected
market-btc-page-actual market-btc-page-diff market-btc-page-expected
app-locked-actual app-locked-diff app-locked-expected
app-locked-actual app-locked-diff app-locked-expected

windows

Actual Diff Expected
market-btc-page-actual market-btc-page-diff market-btc-page-expected
market-btc-page-actual market-btc-page-diff market-btc-page-expected
app-locked-actual app-locked-diff app-locked-expected
app-locked-actual app-locked-diff app-locked-expected

@valpinkman valpinkman merged commit 858effd into monorepo-setup May 3, 2022
@valpinkman valpinkman deleted the feat/more-ci-reporting branch May 3, 2022 15:14
live-github-bot bot pushed a commit that referenced this pull request Oct 3, 2023
@davidecarpini
Merge pull request #33 from vechainfoundation/carpini/merge-2023-10-02-2
5f1d02a 
Carpini/merge 2023 10 02 2
VicAlbr added a commit that referenced this pull request Jul 16, 2024
@VicAlbr
# This is a combination of 38 commits.
1c42609 
# This is the 1st commit message:

wip: adding Xray reporter for speculos tests

# The commit message #2 will be skipped:

# wip: xray reporter

# The commit message #3 will be skipped:

# wip: Xray - Speculos tests

# The commit message #4 will be skipped:

# removing xray package

# The commit message #5 will be skipped:

# Xray with Json

# The commit message #6 will be skipped:

# rebase + merge conflict

# The commit message #7 will be skipped:

# test: adding the possibility to send resultats to Xray

# The commit message #8 will be skipped:

# test: code cleanning

# The commit message #9 will be skipped:

# ci: update workflow

# The commit message #10 will be skipped:

# ci: remove workflow input

# The commit message #11 will be skipped:

# ci: workflow

# The commit message #12 will be skipped:

# ci: adding logs

# The commit message #13 will be skipped:

# test: update code after review

# The commit message #14 will be skipped:

# test: removing test_summary + update worflow + adding logging response data in error throw

# The commit message #15 will be skipped:

# fix: resolved conflicts

# The commit message #16 will be skipped:

# test: update xray method + Json reporter

# The commit message #17 will be skipped:

# remove com

# The commit message #18 will be skipped:

# update tmsLink

# The commit message #19 will be skipped:

# code cleanning

# The commit message #20 will be skipped:

# ci: print path

# The commit message #21 will be skipped:

# creating file path

# The commit message #22 will be skipped:

# no test execution

# The commit message #23 will be skipped:

# update workflow

# The commit message #24 will be skipped:

# resolve workflow error

# The commit message #25 will be skipped:

# workflow env update

# The commit message #26 will be skipped:

# ci: Xray_Base_Url update

# The commit message #27 will be skipped:

# update path for xray reporter

# The commit message #28 will be skipped:

# ci: logging response status

# The commit message #29 will be skipped:

# update xray authenticate

# The commit message #30 will be skipped:

# ci: publish to Xray

# The commit message #31 will be skipped:

# update playwright config + print xray link on CI logs

# The commit message #32 will be skipped:

# update workflow

# The commit message #33 will be skipped:

# ci: update json file

# The commit message #34 will be skipped:

# ci: logging files for debuug

# The commit message #35 will be skipped:

# ci: merged json file

# The commit message #36 will be skipped:

# ci: adding script to merge Xray payload

# The commit message #37 will be skipped:

# ci: update workflow - run

# The commit message #38 will be skipped:

# autho Permission
live-github-bot bot pushed a commit that referenced this pull request Sep 3, 2024
@popenta
Merge pull request #33 from multiversx/merge-develop-03-09
d17d997 
Another merge from develop
hedi-edelbloute pushed a commit that referenced this pull request Oct 1, 2024
@pavanvora
Merge pull request #33 from mehulcs/tx-errors-warnings
8ad5b5f 
show tx status warnings and errors
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@elbywan elbywan elbywan approved these changes

Assignees

@valpinkman valpinkman

Labels
automation CI/CD stuff tools Has changes in tools
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@valpinkman @elbywan