Skip to content

[ci] Attempt to fix post-merge builds on main#12210

Merged
driazati merged 1 commit intoapache:mainfrom
areusch:fix-git-change-docker
Jul 28, 2022
Merged

[ci] Attempt to fix post-merge builds on main#12210
driazati merged 1 commit intoapache:mainfrom
areusch:fix-git-change-docker

Conversation

@areusch
Copy link
Contributor

@areusch areusch commented Jul 28, 2022

  • This patch addresses [ci] Docker builds on main aren't triggering #12097 by referencing the branch from the BRANCH_NAME environment variable, which is also used in the Jenkinsfile
  • This avoids git rev-parse, which assumes the local repo contains a git branch that matches the name of the branch being merged. I think this is in spirit of what the script was trying for.

cc @Mousius @driazati @yongwww

@github-actions github-actions bot requested a review from driazati July 28, 2022 00:17
@driazati driazati changed the title Attempt to fix post-merge builds on main [ci] Attempt to fix post-merge builds on main Jul 28, 2022
@github-actions
Copy link
Contributor

Built docs for commit 8ab57ea can be found here.

@driazati driazati merged commit 0674858 into apache:main Jul 28, 2022
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
* This patch addresses apache#12097 by referencing the branch from the `BRANCH_NAME` environment variable, which is also used in the Jenkinsfile
* This avoids git rev-parse, which assumes the local repo contains a git branch that matches the name of the branch being merged. I think this is in spirit of what the script was trying for.
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