Skip to content

Refactor release notes handling in workflow#24830

Merged
m-aliozkaya merged 1 commit intodevfrom
skoc10-patch-1
Feb 6, 2026
Merged

Refactor release notes handling in workflow#24830
m-aliozkaya merged 1 commit intodevfrom
skoc10-patch-1

Conversation

@skoc10
Copy link
Contributor

@skoc10 skoc10 commented Feb 6, 2026

No description provided.

Updated the workflow to save release notes to an environment variable instead of a temporary file. Adjusted validation and fallback mechanisms accordingly.
@m-aliozkaya m-aliozkaya merged commit 512faa7 into dev Feb 6, 2026
5 of 6 checks passed
@m-aliozkaya m-aliozkaya deleted the skoc10-patch-1 branch February 6, 2026 12:50
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR refactors how release notes are handled in the GitHub Actions workflow by switching from file-based storage to environment variable storage. The change eliminates the need for a temporary file (.tmp/raw-notes.txt) in earlier steps while maintaining the same functionality.

Changes:

  • Replaced file-based release notes storage with environment variable (RAW_NOTES) using heredoc syntax
  • Updated validation logic to check environment variable instead of file existence
  • Modified AI prompt and fallback processing to use the environment variable directly
Comments suppressed due to low confidence (1)

.github/workflows/update-studio-docs.yml:1

  • The pipe to while creates a subshell, which means the output redirection to .tmp/final-notes.txt happens in the subshell. This could result in an empty or incomplete file. Consider using process substitution (while IFS= read -r line; do ... done < <(echo \"$RAW_NOTES\")) or a here-string (while IFS= read -r line; do ... done <<< \"$RAW_NOTES\") instead.
name: Update ABP Studio Docs

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.

3 participants