Skip to content
This repository has been archived by the owner on Dec 6, 2023. It is now read-only.

minor CI improvements #78

Merged
merged 3 commits into from
Jul 28, 2023
Merged

minor CI improvements #78

merged 3 commits into from
Jul 28, 2023

Conversation

casperdcl
Copy link
Contributor

@casperdcl casperdcl commented Jul 28, 2023 •
edited
Loading

  • don't report failure when there's nothing to do (no models changed)
  • fix teardown permissions
  • fix docker push permission error

follow-up to #75 <- #70

@casperdcl casperdcl added bug Something isn't working ci CI/CD labels Jul 28, 2023
@casperdcl casperdcl self-assigned this Jul 28, 2023
@casperdcl casperdcl temporarily deployed to internal July 28, 2023 14:32 — with GitHub Actions Inactive
@casperdcl casperdcl temporarily deployed to internal July 28, 2023 15:31 — with GitHub Actions Inactive
@filopedraz filopedraz merged commit eb075c9 into premAI-io:main Jul 28, 2023
3 checks passed
@casperdcl casperdcl deleted the ci-convenience branch July 28, 2023 17:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@filopedraz filopedraz filopedraz approved these changes

Assignees

@casperdcl casperdcl

Labels
bug Something isn't working ci CI/CD
Projects
No open projects
Prem ML
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@casperdcl @filopedraz