Skip to content

Allow .artifacts for changelog CI I/O - #3913

Merged
lcawl merged 1 commit into
mainfrom
changelog-hidden-directories
Aug 24, 2026
Merged

Allow .artifacts for changelog CI I/O#3913
lcawl merged 1 commit into
mainfrom
changelog-hidden-directories

Conversation

@lcawl

@lcawl lcawl commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Why

changelog add --output .artifacts/changelog-staging (used by docs-actions changelog-submit) still fails when generation runs: ChangelogFileSystem rejects hidden .artifacts path segments. #3917 already fixed the overlapping prepare-artifact / RunnerTempFileSystem path.

Also requires elastic/docs-actions#308 so upload-artifact includes files under .artifacts/changelog-artifact.

Test plan

  • dotnet test tests/Elastic.Changelog.Tests/
  • After edge publish: confirm a changelog-submit run that does generate still succeeds end-to-end

What

  • Allowlist .artifacts on ChangelogFileSystem (aligned with other scopes that already allow it).
  • Add focused filesystem regression tests: allow .artifacts, block other hidden dirs.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No actionable issues found. Verdict: approve.


What is this? | From workflow: PR Review

Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.

@lcawl

lcawl commented Aug 24, 2026

Copy link
Copy Markdown
Contributor Author

Rebase analysis:

@lcawl
lcawl force-pushed the changelog-hidden-directories branch from 7d0b3cd to a1297ce Compare August 24, 2026 19:21
@lcawl
lcawl enabled auto-merge (squash) August 24, 2026 19:26
@lcawl
lcawl merged commit b5f3694 into main Aug 24, 2026
31 checks passed
@lcawl
lcawl deleted the changelog-hidden-directories branch August 24, 2026 19:32
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