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

[#190] Fix usePlayer and related hooks to not rely on instrument order and their indexes #200

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

michalsek
Copy link
Member

Closes #190

Introduced changes

  • Replaces playingNotes with instrumentName -> boolean map

Checklist

  • Linked relevant issue
  • Updated relevant documentation
  • Added/Conducted relevant tests
  • Performed self-review of the code

@michalsek michalsek self-assigned this Nov 6, 2024
apps/common-app/src/examples/DrumMachine/PlayButton.tsx Dismissed Show dismissed Hide dismissed
apps/fabric-example/ios/Podfile.lock Dismissed Show dismissed Hide dismissed
Copy link

github-actions bot commented Nov 6, 2024

@check-spelling-bot Report

🔴 Please review

See the 📂 files view, the 📜action log, or 📝 job summary for details.

Unrecognized words (574)
aabe
aae
aarch
abe
abecc
abefimoux
abffd
ABGJMOPRSUW
ACDD
ADACE
Additonal
aead
aec
aecba
afe
afffb
akx
allprojects
androiddebugkey
androidx
apk
appname
appveyor
ARCHS
arget
armeabi
arnings
arounds
asciinema
ASingle
askubuntu
assetcatalog
assword
ath
attributedstring
audioapi
Autolayout
autoreleasepool
AUTORELEASING
autoresizing
ayb
baccounts
badgen
baf
bamazon
bany
bapi
bapp
barchive
basciinema
bbadgen
bbcda
BBD
bbit
bbitbucket
bbootstrapcdn
bca
bcalendar
bcb
bcbb
bcdlpsw
bcdnjs
bchrome
bci
BCIPR
bcircleci
bclients
bcodepen
bcolab
bcompai
Bcu
bdatastudio
bdeveloper
bdisqus
bdocs
bdrive
bdropbox
bembed
bfa
bfb
bfbcdn
bfburl
bfcc
bfd
bfe
BFF
bffcf
bffdfd
bfigma
bfit
bfonts
bforms
bfreecodecamp
bgetopts
bgist
bgit
bgithub
bgithubassets
bgitlab
bgitter
bgnupg
bgoogle
bgoogleapis
bgravatar
bgreater
bgroups
bimage
bimg
bimgur
binance
bitbucket
BITCODE
bitrise
blogs
bmaps
bmastodon
bmedia
bmedium
bmermaid
bmicrosoft
bmount
bmusic
bmvnrepository
BNDL
bneither
bno
bnon
bnor
bootstrapcdn
bopen
Bosa
bpastebin
bpm
bqr
breddit
bregex
bregistry
breif
brequestb
bscastie
bscholar
bset
bshields
bslack
bsns
bstorage
bstruct
btinyurl
bto
btput
btwimg
btwitter
btype
buckd
Buildables
buildscript
bvisualstudio
bwork
bxn
caad
cac
cae
callinvoker
CBA
cbb
CBBA
cbbf
CBCC
CBlock
cca
ccache
ccb
ccc
ccce
cce
ccfcbd
cdc
cdcd
cddb
CDeclaration
cdn
CDPATH
cea
cefac
cefbd
cer
cfbfc
cfbundle
circleci
cloudflare
cocoapods
codacy
codepen
Colab
commitlint
compai
componentregistry
componentregistrynative
contributorsrc
cosw
coveragerc
CPLUSPLUSFLAGS
cpplint
crt
CSpace
cstddef
CStyle
cxxreact
Cyg
cygpath
dadb
DANDROID
datasources
dbbc
dbd
dbe
dbfef
DCACB
ddb
ddbc
ddf
ddfeb
debuggable
defaultsnativemodule
detials
dfc
DFOLLY
diffs
dimen
Disqus
diuoxn
domnativemodule
Dorg
dotnet
doxygen
DPL
DPWXY
drawio
DRCT
DREACT
dropbox
dsprelated
eaa
EABD
eacba
eba
ebaab
ebaf
ebfba
ecae
ecent
edb
edc
EEFC
efc
efcf
efefef
EHf
emmintrin
emplates
emptively
entrancy
eport
epreciation
equ
esolve
esult
evilmartians
faaaa
fabricexample
fadfb
faf
FBC
fbcef
FBD
fbe
FBF
fbjni
fca
fccb
fdb
fdc
fdce
FDDC
fde
featureflags
featureflagsnativemodule
febf
fef
fega
fexceptions
ffac
FFD
ffe
ffffff
FFFFFFF
figma
fprime
freebooks
freecodecamp
frtti
fstack
ftu
GETTIME
GHSA
giphy
githubassets
Gitlab
gitweb
GJd
gle
glog
googleusercontent
gpg
gradlew
graffle
gstatic
hashtag
hermesc
heroku
hmap
IAudio
IBiquad
icns
idlecallbacks
idlecallbacksnativemodule
IGain
IIR
imagemanager
imgur
ine
INFOPLIST
inputaccessory
instancetype
Instantitation
IOscillator
iostextinput
ipa
ipfs
iphoneos
iphonesimulator
ipynb
IRR
IService
IStereo
itle
IWYU
JAVACMD
javaobject
jetbrains
jhybriddata
jhybridobject
jks
JMessage
jni
JNICALL
JNIEXPORT
jpe
jre
jsbundle
jsc
jserrorhandler
jsi
jsiexecutor
jsinspector
jsitracing
jsoref
jvmargs
jzt
keystore
kotlin
kotlinx
ksh
ktlint
ktx
Lcom
LDFLAGS
leakchecker
lefthook
legacyviewmanagerinterop
LIBCPP
libfbjni
libfolly
libhermes
libjscexecutor
libjsi
libreactnativejni
libreanimated
LICEN
linelength
Llf
Llpsw
loadu
LOCALIZABILITY
LStv
mailgun
Mapbuffer
maxmgv
mcve
metacharacter
microtasksnativemodule
mozilla
MSGSEND
msys
MTL
mtrl
mvnrepository
nativeconfig
nativemodule
ndk
ndkversion
necho
nfi
nif
nistp
NONINFRINGEMENT
npmjs
nrt
NSURL
NYQUIST
objc
objcpp
ocf
OMo
Onone
openpgpkey
ormal
osub
otf
OZV
pastebin
pbxproj
pbxuser
Pdocker
perflogger
performancetimeline
perspectivev
pgp
phc
pinterest
pki
pne
podfile
podspec
posthog
postpack
powerbi
prama
proguard
pylintrc
QParam
qscriptbuffer
qtproject
qtscriptclassic
rdparty
reactjs
reactnative
reactnativejni
reactperflogger
redbox
regexplanet
regularbeat
rendererconsistency
rendererdebug
repostiory
reproducting
requestb
retvl
rncore
RNv
rootproject
rubygems
runtimeexecutor
Runtimes
runtimescheduler
rvm
RWb
safeareaview
Samsung
scastie
screenshots
scrollview
SDKROOT
serverfault
setzero
sfor
shellness
shopify
skia
SLO
soloader
sopt
sourcecode
spotify
SRCROOT
Srcs
Ssx
stackexchange
stackoverflow
statuspage
stereopanner
storeu
STREQUAL
subviews
svgz
swm
swmansion
swork
tada
taskdefs
templateprocessor
templatesz
terminfo
Testables
textlayoutmanager
tgz
thewritepractice
tinyurl
tmdb
tprteaching
troff
Ttx
turbomodule
Turborepo
uimanager
unimplementedview
unsplash
usercontent
userguide
utm
vabsq
vadd
vaddq
vdupq
vget
vld
vmax
vmaxq
vmul
vmulq
vsadd
vsmul
vsub
vsubq
webaudio
webstore
Werror
Wno
workarounds
worklets
WWRY
xcassets
xccheckout
xcconfig
xcf
xcfilelist
xcodeproj
xcodeproject
xcprivacy
XCT
xctest
xcuserdata
xcuserstate
xcworkspace
XDE
Xms
Xmx
xpm
XYWH
Ylf
youtu
youtube
youtubei
Zacdfh
Some files were automatically ignored 🙈

These sample patterns would exclude them:

[^/]\.png$
^\Q.github/actions/setup/spelling/allow.txt\E$
^\Q.github/actions/setup/spelling/expect.txt\E$
^\Q.yarn/releases/yarn-4.5.0.cjs\E$
^\Qapps/fabric-example/android/app/debug.keystore\E$
^\Qapps/fabric-example/android/gradle/wrapper/gradle-wrapper.jar\E$
^\Qyarn.lock\E$

You should consider excluding directory paths (e.g. (?:^|/)vendor/), filenames (e.g. (?:^|/)yarn\.lock$), or file extensions (e.g. \.gz$)

You should consider adding them to:

.github/actions/spelling/excludes.txt

File matching is via Perl regular expressions.

To check these files, more of their words need to be in the dictionary than not. You can use patterns.txt to exclude portions, add items to the dictionary (e.g. by adding them to allow.txt), or fix typos.

To accept these unrecognized words as correct and update file exclusions, you could run the following commands

... in a clone of the [email protected]:software-mansion-labs/react-native-audio-api.git repository
on the fix/dm-instrument-order branch (ℹ️ how do I use this?):

curl -s -S -L 'https://raw.githubusercontent.com/check-spelling/check-spelling/main/apply.pl' |
perl - 'https://github.com/software-mansion-labs/react-native-audio-api/actions/runs/11701922165/attempts/1'

OR

To have the bot accept them for you, comment in the PR quoting the following line:
@check-spelling-bot apply updates.

Available 📚 dictionaries could cover words not in the 📘 dictionary
Dictionary Entries Covers Uniquely
cspell:cpp/src/compiler-clang-attributes.txt 46 2 1
cspell:cryptocurrencies/cryptocurrencies.txt 125 2 2
cspell:cpp/src/ecosystem.txt 51 1 1
cspell:ruby/dict/ruby.txt 157 1 1
cspell:css/dict/css.txt 263 1 1

Consider adding them (in .github/workflows/spelling.yml) in jobs:/spelling: for uses: check-spelling/check-spelling@main in its with:

      with:
        extra_dictionaries: |
          cspell:cpp/src/compiler-clang-attributes.txt
          cspell:cryptocurrencies/cryptocurrencies.txt
          cspell:cpp/src/ecosystem.txt
          cspell:ruby/dict/ruby.txt
          cspell:css/dict/css.txt

To stop checking additional dictionaries, add (in .github/workflows/spelling.yml) for uses: check-spelling/check-spelling@main in its with:

check_extra_dictionaries: ''
Errors (4)

See the 📂 files view, the 📜action log, or 📝 job summary for details.

❌ Errors Count
⚠️ binary-file 15
⚠️ large-file 1
⚠️ noisy-file 1
❌ unsupported-repo-notation 2

See ❌ Event descriptions for more information.

@michalsek michalsek merged commit 3280750 into main Nov 6, 2024
9 of 11 checks passed
@michalsek michalsek deleted the fix/dm-instrument-order branch November 6, 2024 13: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.

[Example - Drum Machine] Fix usePlayer and related hooks to not rely on instrument order and their indexes
2 participants