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

[pre-commit] Add DOCKERFILES_INFO_REPO_PRIMARY_BRANCH #4325

Merged
merged 3 commits into from
Jun 5, 2024

Conversation

mmhw
Copy link
Contributor

@mmhw mmhw commented Jun 5, 2024

Description

Customers running pre-commit and having the DEMISTO_DEFAULT_BRANCH environment variable set to "main" for
example receive the following error:

FileNotFoundError: file in https://raw.githubusercontent.com/demisto/dockerfiles-info/main/docker_images_metadata.json does not exist

In this case it is not necessary to use this variable but simply use master.

@mmhw mmhw self-assigned this Jun 5, 2024
Copy link

github-actions bot commented Jun 5, 2024

Changelog(s) in markdown:

  • Fixed an issue in the pre-commit command where if the DEMISTO_DEFAULT_BRANCH environment variable was set to something other than "master", the command would fail. #4325

@mmhw mmhw merged commit 28b0552 into master Jun 5, 2024
25 checks passed
@mmhw mmhw deleted the MW/pre-commit/add_DOCKERFILES_INFO_REPO_PRIMARY_BRANCH branch June 5, 2024 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants