Skip to content

feat: update for new file system implementation - #75

Merged
audiostackhenry merged 8 commits into
mainfrom
feat/update-file-v2
Jul 29, 2025
Merged

feat: update for new file system implementation#75
audiostackhenry merged 8 commits into
mainfrom
feat/update-file-v2

Conversation

@harry97uk

Copy link
Copy Markdown

No description provided.

@harry97uk
harry97uk requested a review from CodeBooster97 April 23, 2025 16:31
@qlty-cloud-legacy

Copy link
Copy Markdown

❌ 5 blocking issues (5 total)

Tool Category Rule Count
radarlint-python Lint Replace this generic exception class with a more specific one. 3
radarlint-python Lint Rename this parameter "fileName" to match the regular expression ^[_a-z][a-z0-9_]*$. 1
radarlint-python Lint Define a constant instead of duplicating this literal "https://v2\.api\.audio/v3" 9 times. 1

This is from Qlty Cloud, the successor to Code Climate Quality. Learn more.

Comment thread audiostack/content/file.py
Comment thread audiostack/content/file.py Outdated
Comment thread audiostack/content/file.py Outdated
Comment thread audiostack/content/file.py Outdated
Comment thread audiostack/content/file.py Outdated
@CodeBooster97
CodeBooster97 force-pushed the feat/update-file-v2 branch 7 times, most recently from 52cfd8d to f0a4662 Compare July 25, 2025 16:14
@CodeBooster97 CodeBooster97 changed the title [WIP] feat: update for new file system implementation feat: update for new file system implementation Jul 28, 2025
@CodeBooster97
CodeBooster97 force-pushed the feat/update-file-v2 branch 2 times, most recently from 079ac11 to 913f099 Compare July 28, 2025 15:46
Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated

@audiostackhenry audiostackhenry left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved except the date on the change log. Please update and then merge

@audiostackhenry
audiostackhenry merged commit 722fb31 into main Jul 29, 2025
1 check passed
audiostackhenry added a commit that referenced this pull request Jul 29, 2025
* version poetry

* feat: add query method to Voice with tests

* updoate

* fix comit

* Fix - remove emojis from section headers and add public asset link

* Include changelog

* bump version

* feat: add customer trace id header

* feat: enhance header creation to accept custom headers

* docs: update changelog to include custom headers feature

* feat: implement context manager for setting customer trace id

* chore: reorganize imports in request_interface.py

* feat: update use_trace context manager to specify return type and add tests for trace id handling

* fix: update use_trace context manager to improve type hinting for trace_id

* fix: improve type hinting for current_trace_id in request_interface.py

* chore: reorder imports in request_interface.py for consistency

* fix: pass 0 float values (#73)

* fix: pass 0 float values

* chore: changelog

* chore: lint

* feat: update for new file system implementation (#75)

* feat: update for new file system implementation
* change base url
* only useful methods
* add docstrings
* fix files and typing
* fix cc reporter
* revert snake_case to camelCase
* Update CHANGELOG.md

* fix: Deleted unused import

---------

Co-authored-by: Guillem <[email protected]>
Co-authored-by: Harry Gardiner <[email protected]>
Co-authored-by: CodeBooster97 <[email protected]>
Co-authored-by: Lars <[email protected]>
Co-authored-by: Marcin Szleszynski <[email protected]>
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.

3 participants