Use 0.29.0 CLI in IDE2 #1683
Merged
Use 0.29.0 CLI in IDE2 #1683
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kittaakos
approved these changes
Nov 17, 2022
kittaakos
added
topic: infrastructure
AlbyIanna
force-pushed
the
cli-0.29.0
branch
from
399318d to
cda8c3b
Compare
3 tasks
per1234
added a commit
that referenced
this pull request
Dec 7, 2022
The "Arduino IDE" GitHub Actions workflow generates a changelog from the commits since the last tag. This changelog is published in multiple ways: - Printed to workflow run logs - Uploaded to Arduino's download server (mostly useful for the nightly builds) - Initial version of release notes For the last, the changelog text must be passed from the dedicated changelog generation workflow step to the release step. This is done via workflow job output. At the time the system was set up, outputs for workflow `run` steps were set using the `set-output` workflow command. That "workflow command" system was later determined by GitHub to have potential security vulnerabilities, so it was replaced with a `GITHUB_OUTPUT` environment file. The "Arduino IDE" workflow was migrated to the new "environment file" approach. It was later discovered that there was an undocumented breaking change in the method for handling multi-line strings in workflow step outputs between the old "workflow command" system and the new "environment file". This resulted in the initial release notes having an incorrect format. For example, what would previously have been formatted like this: - Updated translation files (#1606) [23c7f5f] - Use 0.29.0 CLI in IDE2 (#1683) [f1144ef] Was now formatted like this: - Updated translation files (#1606) [23c7f5f]%0A - Use 0.29.0 CLI in IDE2 (#1683) [f1144ef]%0A The solution is to remove the commands that did the escaping of the changelog text in a manner that is no longer supported and replace them with a "here document"-style format. A random number is used as the "delimiter" (limit string) per the security recommendations in the official GitHub documentation. Note that even though the multiline strings handling documentation was placed under the environment variable section, it also applies to setting outputs.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation
Use the latest (0.29.0) CLI in IDE2.
Change description
Other information
Reviewer checklist