-
Notifications
You must be signed in to change notification settings - Fork 1
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
fix: revert deletion of tests auto-run after pull_request changes #64
base: main
Are you sure you want to change the base?
Conversation
It's not visible in PR #63 yet somehow most of the tests are currently skipped on the main branch. Is it something that is necessary? |
de7ca66
@kat-kan : they should not be skipped - I've just removed the |
@kat-kan , @adamcegielka , @KarolinaZakrzewska , @Justyna-KO : could you please review the changes? Also, please note that I marked 2 tests as fixme from the
|
a152404
# Conflicts: # .github/workflows/pw-tests.yml # tests/api/articles/articles-post.spec.ts
@Justyna-KO @Slawomir-DKl I updated this branch and PR description, the history of branch is overwhelming but eventually the changes not so much :) please review! |
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.
I have one remark only
Resolves #19
Continuation of PR #63
which was merged by accident
Scope of changes
revert deletion of running on pr changes(must have been fixed in another PR, comment by @kat-kan).md
files are modified)PR steps completed:
kawqa-gad-playwright
project's git commit message format