feat(services/compfs): basic Access impl#4693
Conversation
github-actions
bot
added
the
releases-note/feat
George-Miao
changed the title
Access implAccess impl for services-compfs
George-Miao
changed the title
Access impl for services-compfsAccess impl for services-compfs
George-Miao
changed the title
Access impl for services-compfsAccess impl
George-Miao
force-pushed
the
compio-master
branch
2 times, most recently
from
45ee5c6 to
791bb95
Compare
George-Miao
changed the title
Access implAccess impl
George-Miao
force-pushed
the
compio-master
branch
from
1ded7fd to
1a3bf7c
Compare
|
cc @George-Miao, would you like to submit a PR to bump |
Sure. Will do |
I have updated all |
George-Miao
force-pushed
the
compio-master
branch
from
1a3bf7c to
c7b32c7
Compare
|
Hmmm.. compio doesn't support wasm so it won't compile under wasm. Besides that all CI's have passed. @Xuanwo |
Only few services are enabled under wasm target. Maybe we should make all compio related crates optional? |
|
Oh I see. I didn't mark driver as optional. I'll change that later. Do you think it's ok to release new version for compio? After that we can merge this PR. |
Looks nice to me so far. Let's cut a new version for compio 🤩 |
George-Miao
force-pushed
the
compio-master
branch
from
c7b32c7 to
72a96c5
Compare
George-Miao
force-pushed
the
compio-master
branch
from
3bbb253 to
ce40456
Compare
2 participants
This PR implements basic function of
AccessforCompfs, includeCompfsReader,CompfsWriterandCompfsLister.For now, this branch uses git dependency ofcompioto test the lastest updates. Before merging, compio should make a new release (by @George-Miao, me) and update the correspondingCargo.tomlentry.New release has been made and updated. Ready to be merged.