Skip to content

use default docker repo name if variable is not defined #6870

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

Merged

Conversation

AndrewChubatiuk
Copy link
Contributor

What type of PR is this?

Fallback to default DOCKER_REPO env variable in case if DOCKER_REPO actions variable is not defined

  • Refactor
  • Feature
  • Bug Fix
  • New Query Runner (Data Source)
  • New Alert Destination
  • Other

Description

How is this tested?

  • Unit tests (pytest, jest)
  • E2E Tests (Cypress)
  • Manually
  • N/A

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

@AndrewChubatiuk AndrewChubatiuk force-pushed the fallback-docker-repo-name branch from 671212c to 1a5c1dd Compare April 10, 2024 12:30
@justinclift justinclift enabled auto-merge (squash) April 10, 2024 12:44
auto-merge was automatically disabled April 10, 2024 13:32

Head branch was pushed to by a user without write access

@AndrewChubatiuk AndrewChubatiuk force-pushed the fallback-docker-repo-name branch from 1a5c1dd to 7fbd804 Compare April 10, 2024 13:32
@justinclift justinclift merged commit d7247f8 into getredash:master Apr 10, 2024
@AndrewChubatiuk AndrewChubatiuk deleted the fallback-docker-repo-name branch April 10, 2024 21:11
harveyrendell pushed a commit to pushpay/redash that referenced this pull request Jan 8, 2025
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