Skip to content
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

i#6678 Rename aarchxx workflows, part 2: Rename native ci-aarchxx file #6705

Merged
merged 2 commits into from
Mar 13, 2024

Conversation

xdje42
Copy link
Contributor

@xdje42 xdje42 commented Mar 13, 2024

There are currently two workflows name "ci-aarchxx" which is confusing. The ci-aarch64-native workflow was renamed to ci-aarchxx in #6549 with the goal of being consistent with other architecture workflows. However, this introduced the ambiguous "ci-aarchxx" naming.

Fix this by renaming ci-aarchxx.yml -> ci-aarchxx-cross.yml, and renaming ci-aarch64-native.yml -> ci-aarchxx.yml.
Furthermore, rename the aarchxx "cross" workflow to ci-aarchxx-cross.

This accomplishes three things:

  1. Naming is consistent with the other workflow files: native workflows are named "ci-$ARCH" and live in file "ci-$ARCH.yml". 2) All workflows have unique names.
  2. The yml file and workflow have the same name.

In part one (#6696) we renamed the cross ci-aarchxx to ci-aarchxx-cross, in the workflow name and yml file name.
In this part two we leave the workflow name alone, but still rename the file from ci-aarch64-native.yml to ci-aarchxx.yml. The renaming is split into two to better preserve the history: Git can track file renamings, but not when the original file is also replaced with something different.

Fixes #6678

… file

There are currently two workflows name "ci-aarchxx" which is confusing.
The ci-aarch64-native workflow was renamed to ci-aarchxx in #6549 with
the goal of being consistent with other architecture workflows.
However, this introduced the ambiguous "ci-aarchxx" naming.

Fix this by renaming ci-aarchxx.yml -> ci-aarchxx-cross.yml, and
renaming ci-aarch64-native.yml -> ci-aarchxx.yml.
Furthermore, rename the aarchxx "cross" workflow to ci-aarchxx-cross.

This accomplishes three things:
1) Naming is consistent with the other workflow files: native workflows
are named "ci-$ARCH" and live in file "ci-$ARCH.yml".
2) All workflows have unique names.
3) The yml file and workflow have the same name.

In part one (#6696) we renamed the cross ci-aarchxx to ci-aarchxx-cross,
in the workflow name and yml file name.
In this part two we leave the workflow name alone, but still
rename the file from ci-aarch64-native.yml to ci-aarchxx.yml.
The renaming is split into two to better preserve the history:
Git can track file renamings, but not when the original file is also
replaced with something different.

Fixes #6678
@xdje42 xdje42 merged commit 4c8e947 into master Mar 13, 2024
16 checks passed
@xdje42 xdje42 deleted the i6678-rename-ci-aarchxx-native-workflow-2 branch March 13, 2024 21:05
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.

INFRASTRUCTURE: Two workflows named ci-aarchxx
2 participants