Skip to content
Snippets Groups Projects

ci: Fix publishing to pages from default branch

Merged Ritesh Raj Sarraf requested to merge wip/ritesh/use-rules-default-branch.master into master

ci: Fix publishing to pages from default branch

Commit 728c9142 "Use CI_DEFAULT_BRANCH as the default branch" actually broke the publishing via GitLab pages from the default branch since the only: [ ${CI_DEFAULT_BRANCH} ] condition was not really expanding the variable and it was considering ${CI_DEFAULT_BRANCH} as a literal branch name.

This means that the test-pdf job got always instantiated while pages never was, regardless of the branch they where running on.

By switching to rules: we can properly implement automatic default branch detection and fix the regression.

Signed-off-by: Ritesh Raj Sarraf ritesh.sarraf@collabora.com

Edited by Ritesh Raj Sarraf

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Emanuele Aina mentioned in merge request !617 (merged)

    mentioned in merge request !617 (merged)

    • Resolved by Ritesh Raj Sarraf

      The commit message is quite bad tbh. It is stating the "what" and not explaining the "why". I mean, it should definitely mention that this is a fix for a regression introduced in the previous commit, but there's no indication of that.

  • added 1 commit

    Compare with previous version

  • added 1 commit

    • 6851c940 - Enable the jobs to run conditionally

    Compare with previous version

  • added 2 commits

    • ebea9c5d - 1 commit from branch master
    • 5f8e40bd - Enable the jobs to run conditionally

    Compare with previous version

  • added 1 commit

    • 72ba8399 - Enable the jobs to run conditionally

    Compare with previous version

  • added 2 commits

    • ebea9c5d - 1 commit from branch master
    • 4f9bc55f - Enable the jobs to run conditionally

    Compare with previous version

  • Ritesh Raj Sarraf requested review from @em

    requested review from @em

  • added 1 commit

    • 7c062841 - ci: Fix publishing to pages from default branch

    Compare with previous version

  • Ritesh Raj Sarraf changed title from Run under condition rules to ci: Fix publishing to pages from default branch

    changed title from Run under condition rules to ci: Fix publishing to pages from default branch

  • Ritesh Raj Sarraf changed the description

    changed the description

  • added 3 commits

    Compare with previous version

  • Ritesh Raj Sarraf resolved all threads

    resolved all threads

  • Dylan Aïssi mentioned in merge request !621 (merged)

    mentioned in merge request !621 (merged)

  • added 2 commits

    • 7497bfe5 - 1 commit from branch master
    • 62e2b909 - ci: Fix publishing to pages from default branch

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading