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

cml-pr prs list #542

Merged
merged 2 commits into from
May 20, 2021
Merged

cml-pr prs list #542

merged 2 commits into from
May 20, 2021

Conversation

DavidGOrtega
Copy link
Contributor

Something has changed in GH API PRs output that the branch names now includes refs.

  • Fixes the possibility or repeat cml-pr execution retuning existing pr
  • No useless try
  • Removed checkout back (made cml-pr to be always used after dvc and report creation)

@DavidGOrtega DavidGOrtega temporarily deployed to test-internal May 20, 2021 00:19 Inactive
@DavidGOrtega DavidGOrtega temporarily deployed to test-internal May 20, 2021 00:21 Inactive
@DavidGOrtega DavidGOrtega changed the title hotfix/cml-pr prs list cml-pr prs list May 20, 2021
@DavidGOrtega DavidGOrtega self-assigned this May 20, 2021
@DavidGOrtega DavidGOrtega added the p0-critical Max priority (ASAP) label May 20, 2021
@DavidGOrtega DavidGOrtega changed the base branch from master to release/v0.4.1 May 20, 2021 09:55
@DavidGOrtega DavidGOrtega merged commit aecf437 into release/v0.4.1 May 20, 2021
@DavidGOrtega DavidGOrtega deleted the cml-pr-no-B branch May 20, 2021 09:56
DavidGOrtega added a commit that referenced this pull request May 20, 2021
* Fixes fake cml repo for bitbucket publish (#543)

* Fixes fake cml repo for bitbucket publish

* lazy driver

* cml-pr prs list (#542)

* cml-pr no branch sha

* no checkout back and prs fix

* checks if await is ok (#537)

* Rename --cloud-gpu tesla to v100 (#540)

* Add --cloud-gpu v100 option to cml-runner

* Add deprecation message for "tesla" GPU

* 0.4.1

Co-authored-by: Helio Machado <0x2b3bfa0+git@googlemail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p0-critical Max priority (ASAP)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant