Skip to content

Add content-source matching functionality to the assembler #1229

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
merged 1 commit into from
May 8, 2025

Conversation

Mpdreamz
Copy link
Member

@Mpdreamz Mpdreamz commented May 8, 2025

Introduce logic to match branches or tags against configured content sources. Extend CLI with 'content-source' commands and integrate GitHub action for content-source validation. Update project solution and configuration accordingly.

The idea being we can follow up and add

  metadata:
    if: github.event.repository.fork == false && contains(fromJSON('["push"]'), github.event_name)
    concurrency:
      group: ${{ github.workflow }}-${{ github.ref }}
    runs-on: ubuntu-latest
    steps:

      - name: 'Match'
        uses: elastic/docs-builder/actions/assembler-match@main
        with:
          repository: ${{ github.repository }}
          # only works for push
          ref: ${{ github.ref_name }}

As an additional step to the preview workflow while widening up our push triggers.

Example:

env GITHUB_ACTIONS=true dotnet run --project src/tooling/docs-assembler/ -- content-source match docs-content main

info ::e.d.t.f.topwatchFilter:: content-source match :: Starting...
info ::d.a.c.ntSourceCommands:: 'docs-content' 'main' is configured as 'current' content-source

::set-output name=content-source-match::true

::set-output name=content-source-name::current


        0 Errors / 0 Warnings / 0 Hints

info ::e.d.t.f.topwatchFilter:: content-source match :: Finished in '00:00:00.0906530'

Introduce logic to match branches or tags against configured content sources. Extend CLI with 'content-source' commands and integrate GitHub action for content-source validation. Update project solution and configuration accordingly.
@Mpdreamz Mpdreamz requested a review from a team as a code owner May 8, 2025 13:39
@Mpdreamz Mpdreamz self-assigned this May 8, 2025
@Mpdreamz Mpdreamz merged commit 7941c79 into main May 8, 2025
15 checks passed
@Mpdreamz Mpdreamz deleted the feature/content-source-assembler-filter branch May 8, 2025 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants