-
Notifications
You must be signed in to change notification settings - Fork 130
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
Implement checking Dolby Atmos #1275
Implement checking Dolby Atmos #1275
Conversation
Nice thank you! Just some minor nitpicks:
|
Hello, Thanks for review.
If it is not problem for you, you could add this code. |
Done! While checking deeper into this, I also made some small adjustments:
|
Thanks, looks good. |
I can see SonarCloud analysis fails. How can we fix this? |
Yes, that's not much of an issue, just that the sonarcloud key is only linked to this project, not forks, or something like that. It's not blocking its merge as there's another sonarcloud check before an official release anyway. I just didn't merge it yet because I did not take the time yet to properly check that everything is right here, but I think I'll be able to do so this week. As for the corresponding release, it may be done in next weeks. So I guess I will take time to backport those this week, and try making the release in the weeks coming after. In any case, we also now have sort of release previews where we regularly publish on npm pre-releases under the |
Implement checking Dolby Atmos
Implement checking Dolby Atmos
Implement checking Dolby Atmos
Implement checking Dolby Atmos
Implement checking Dolby Atmos
Implement checking Dolby Atmos
Implement checking Dolby Atmos
Implement checking Dolby Atmos
I have implemented checking Dolby Atmos in currect representation. #1266