Skip to content

Capitalise “AArch” properly #11047

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 9, 2025
Merged

Capitalise “AArch” properly #11047

merged 1 commit into from
Jul 9, 2025

Conversation

@ffaf1
Copy link
Collaborator

@ffaf1 ffaf1 commented Jul 6, 2025

The official capitalisation is “AArch”,
see https://developer.arm.com/documentation/102374/latest/.

As noted by @ulysses4ever in #11033. Addition in #10705 by @GulinSS (who might want to tell us if he is OK with the change)

  • Patches conform to the coding conventions.
  • Any changes that could be relevant to users have been recorded in the changelog. ← note, forward-porting changelogs to master will have some conflicts because of this.
  • The documentation has been updated, if necessary.
  • Manual QA notes have been included. N/A
  • Tests have been added. (Ask for help if you don’t know how to write them! Ask for an exemption if tests are too complex for too little coverage!) N/A

ffaf1
ffaf1 commented Jul 6, 2025
View reviewed changes
@ffaf1 ffaf1 added the release label Jul 6, 2025
@ffaf1 ffaf1 mentioned this pull request Jul 6, 2025
Merged
2 tasks
ulysses4ever
ulysses4ever approved these changes Jul 6, 2025
View reviewed changes
Copy link
Collaborator

@ulysses4ever ulysses4ever left a comment

Choose a reason for hiding this comment

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

Thank you!

geekosaur
geekosaur approved these changes Jul 6, 2025
View reviewed changes
Copy link
Collaborator

@geekosaur geekosaur left a comment

Choose a reason for hiding this comment

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

Are we fine with the API change? Granting that it seems to have been introduced with the Win/AArch64 changes, so hopefully they haven't really propagated yet.

@ulysses4ever
Copy link
Collaborator

ulysses4ever commented Jul 6, 2025

@geekosaur It's not an API change because this feature has not been released

ffaf1 added a commit to ffaf1/cabal that referenced this pull request Jul 6, 2025
@ffaf1
Add changelog files for 3.16
dc96a96 
* Integrate Brandon’s suggestions.
* Integrate Brandon’s suggestions /II.
* Integrate Bodigrim’s suggestions.
* Integrate Artem’s suggestions.
  * Correctly capitalize “AArch” in preparation of haskell#11047 backport
@ffaf1 ffaf1 added attention: needs-backport 3.16 merge me Tell Mergify Bot to merge labels Jul 7, 2025
@mergify mergify bot added the ready and waiting Mergify is waiting out the cooldown period label Jul 7, 2025
ffaf1 added a commit to ffaf1/cabal that referenced this pull request Jul 8, 2025
@ffaf1
Add changelog files for 3.16
329631d 
* Integrate Brandon’s suggestions.
* Integrate Brandon’s suggestions /II.
* Integrate Bodigrim’s suggestions.
* Integrate Artem’s suggestions.
  * Correctly capitalize “AArch” in preparation of haskell#11047 backport
* Integrate Artem’s suggestions /II.
mergify bot pushed a commit that referenced this pull request Jul 8, 2025
@ffaf1
Add 3.16 changelogs (#11033)
8b69517 
* Add changelog files for 3.16

* Integrate Brandon’s suggestions.
* Integrate Brandon’s suggestions /II.
* Integrate Bodigrim’s suggestions.
* Integrate Artem’s suggestions.
  * Correctly capitalize “AArch” in preparation of #11047 backport
* Integrate Artem’s suggestions /II.

* Remove changelog.d bits
mergify bot pushed a commit that referenced this pull request Jul 8, 2025
@ffaf1 @mergify
Add 3.16 changelogs (#11033)
9a9a16e 
* Add changelog files for 3.16

* Integrate Brandon’s suggestions.
* Integrate Brandon’s suggestions /II.
* Integrate Bodigrim’s suggestions.
* Integrate Artem’s suggestions.
  * Correctly capitalize “AArch” in preparation of #11047 backport
* Integrate Artem’s suggestions /II.

* Remove changelog.d bits

(cherry picked from commit 8b69517)
@mergify mergify bot added the merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days label Jul 9, 2025
mergify bot added a commit that referenced this pull request Jul 9, 2025
@mergify
Merge of #11047
336850e
@mergify mergify bot mentioned this pull request Jul 9, 2025
Closed
86 tasks
@mergify mergify bot merged commit 0278910 into haskell:master Jul 9, 2025
54 checks passed
@ffaf1
Copy link
Collaborator Author

ffaf1 commented Jul 9, 2025

@mergify backport 3.16

@mergify
Copy link
Contributor

mergify bot commented Jul 9, 2025 •
edited
Loading

backport 3.16

✅ Backports have been created

@ffaf1 ffaf1 deleted the aarch-capitalise branch July 9, 2025 11:09
@mergify mergify bot mentioned this pull request Jul 9, 2025
Closed
5 tasks
@ffaf1 ffaf1 mentioned this pull request Jul 9, 2025
Open
38 tasks
mergify bot added a commit that referenced this pull request Jul 9, 2025
@mergify
Merge pull request #11057 from ffaf1/backport-aarch
d5d073d 
Backport #11047: Capitalise “AArch” properly
ffaf1 added a commit to ffaf1/cabal that referenced this pull request Jul 10, 2025
@ffaf1
Add 3.16 changelogs (haskell#11033)
452eead 
* Add changelog files for 3.16

* Integrate Brandon’s suggestions.
* Integrate Brandon’s suggestions /II.
* Integrate Bodigrim’s suggestions.
* Integrate Artem’s suggestions.
  * Correctly capitalize “AArch” in preparation of haskell#11047 backport
* Integrate Artem’s suggestions /II.

* Remove changelog.d bits
@ffaf1 ffaf1 mentioned this pull request Jul 10, 2025
Merged
2 tasks
zlonast pushed a commit to zlonast/cabal that referenced this pull request Aug 25, 2025
@ffaf1 @zlonast
Add 3.16 changelogs (haskell#11033)
a4e2d7b 
* Add changelog files for 3.16

* Integrate Brandon’s suggestions.
* Integrate Brandon’s suggestions /II.
* Integrate Bodigrim’s suggestions.
* Integrate Artem’s suggestions.
  * Correctly capitalize “AArch” in preparation of haskell#11047 backport
* Integrate Artem’s suggestions /II.

* Remove changelog.d bits
zlonast pushed a commit to zlonast/cabal that referenced this pull request Aug 26, 2025
@ffaf1 @zlonast
Add 3.16 changelogs (haskell#11033)
b47fb5c 
* Add changelog files for 3.16

* Integrate Brandon’s suggestions.
* Integrate Brandon’s suggestions /II.
* Integrate Bodigrim’s suggestions.
* Integrate Artem’s suggestions.
  * Correctly capitalize “AArch” in preparation of haskell#11047 backport
* Integrate Artem’s suggestions /II.

* Remove changelog.d bits
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@geekosaur geekosaur geekosaur approved these changes

@ulysses4ever ulysses4ever ulysses4ever approved these changes

+1 more reviewer

@GulinSS GulinSS GulinSS left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

attention: needs-backport 3.16 merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days merge me Tell Mergify Bot to merge ready and waiting Mergify is waiting out the cooldown period release

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ffaf1 @ulysses4ever @GulinSS @geekosaur