template: Change architecture value for Arm AArch64#664
Merged
Conversation
Since aarch is not a standalone term, use either arm or aarch64. Signed-off-by: Andreas Färber <afaerber@suse.com>
Contributor
Author
|
Related: Existing files seem to use |
Contributor
Author
|
Affects #658 file naming. |
This file contains hidden or 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
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.
PR creator: Description
Commit 09d801c updated the template README, referenced from the pull request template, to mandate new files add an architecture prefix of
aarchfor AArch64.Change that mandated value to
arm.It seems it was not yet in use for existing
DC-files. NVIDIA Jetson deployment would be a candidate for adopting it going forward.PR creator: Are there any relevant issues/feature requests?
No.
PR reviewer: Checklist for editorial review
Apart from the usual checks, please double-check also the following: