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

Cypress. Fix/update case 69. #4078

Merged
merged 6 commits into from
Dec 29, 2021

Conversation

dvkruchinin
Copy link
Contributor

Motivation and context

Recently, the test has been falling more often. For this reason, an additional check of the visibility of the drop-down menu has been added.
The test has been finalized and adapted to the current changes in the develop branch. The function of getting the job number has been redesigned. Added state or stage installation functions for jobs.

How has this been tested?

Checklist

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.
  • I have updated the license header for each file (see an example below)
# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

@dvkruchinin dvkruchinin force-pushed the dkru/cypress-fix-case-69 branch from d7dde82 to 5c1eebf Compare December 24, 2021 06:41
@bsekachev
Copy link
Member

bsekachev commented Dec 24, 2021

A lot of CI errors like:

Error: The head commit for this pull_request event is not ahead of the base commit. Please submit an issue on this action's GitHub repo.

maybe merge develop will help?

@dvkruchinin
Copy link
Contributor Author

A lot of CI errors like:

Error: The head commit for this pull_request event is not ahead of the base commit. Please submit an issue on this action's GitHub repo.

maybe merge develop will help?

It seems to have helped. The problem is in action jitterbit/get-changed-files@v1. To solve this problem, I have prepared PR
#4079 (applied the recommendations specified in the actions repository in the issues section).

@bsekachev bsekachev self-assigned this Dec 28, 2021
@bsekachev bsekachev merged commit 7d3116e into cvat-ai:develop Dec 29, 2021
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.

2 participants