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

Update Sony RMT-AA400U Remote Control #570

Merged
merged 2 commits into from
Sep 15, 2023

Conversation

sigv
Copy link
Contributor

@sigv sigv commented Sep 8, 2023

Two parts of the PR regarding Audio_Receivers/Sony/Sony_RMT-AA400U.ir:

  • One removes the executable flag from the file, changing the file mode from 755 to 644.
    These .ir files should not be considered executables.
  • Second applies audio naming scheme changes from commit a68c6a2, which was a part of a large refactoring PR.
    It was merged shortly after the Sony RMT-AA400U IR file was included into the repository. However, the refactoring did not properly consider new files being added. So, Sony RMT-AA400U was not refactored then.

This commit removes the executable flag from Sony_RMT-AA400U.ir,
changing the file mode from 755 to 644. These `.ir` files should
not be considered executables.
Commit a68c6a2 was a part of a large
refactoring PR. It was merged shortly after the Sony RMT-AA400U IR
file was included into the repository. However, the refactoring did not
properly consider new files being added. So, Sony RMT-AA400U was not
refactored. This commit finally applies the audio naming scheme to it.
@UberGuidoZ UberGuidoZ merged commit 15b31ce into Lucaslhm:main Sep 15, 2023
1 check passed
@UberGuidoZ
Copy link
Collaborator

Greatly appreciate this help!

@sigv sigv deleted the update/sony_rmt-aa400u/2023-09 branch September 15, 2023 18:15
UberGuidoZ added a commit that referenced this pull request Jan 19, 2024
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.

2 participants