Skip to content

Add info comment to preview build - #1667

Merged
Mpdreamz merged 3 commits into
mainfrom
feature/add-info-comment
Aug 26, 2025
Merged

Mpdreamz merged 3 commits into
mainfrom
feature/add-info-comment

Conversation

@reakaleek

@reakaleek reakaleek commented Aug 1, 2025 •

Copy link
Copy Markdown
Member

Context

As an attempt to centralize elastic/elasticsearch#132257

Changes

Add a step to the preview build that comments about versioning.

Usage

name: docs-build

on:
  push:
    branches:
      - main
  pull_request_target: ~
  merge_group: ~

jobs:
  docs-preview:
    uses: elastic/docs-builder/.github/workflows/preview-build.yml@main
    with:
      path-pattern: docs/**
      path-pattern-ignore: docs/changelog/**/*.yaml
+     enable-cumulative-comment: true
    permissions:
      deployments: write
      id-token: write
      contents: read
      pull-requests: write

@reakaleek
reakaleek requested a review from a team as a code owner August 1, 2025 12:11
@reakaleek reakaleek self-assigned this Aug 1, 2025
@reakaleek reakaleek added the automation packaging, ci/cd. label Aug 1, 2025
@reakaleek
reakaleek requested a review from leemthompo August 1, 2025 12:11

@Mpdreamz Mpdreamz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It'd be nice if we can rename disable-comments to disable-changes-comment, might be hard unless we support both for a while.

Comment thread .github/workflows/preview-build.yml Outdated
@reakaleek
reakaleek requested a review from Mpdreamz August 4, 2025 19:57
Comment thread .github/workflows/preview-build.yml
Comment thread .github/workflows/preview-build.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation packaging, ci/cd.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants