Skip to content

Async files manipulation using anyio#985

Merged
NakoGH merged 13 commits into
mainfrom
anyio
Jun 5, 2026
Merged

Async files manipulation using anyio#985
NakoGH merged 13 commits into
mainfrom
anyio

Conversation

@armanddidierjean

Copy link
Copy Markdown
Member

Description

Summary

Classification

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 🔨 Refactor (non-breaking change that neither fixes a bug nor adds a feature)
  • 🔧 Infra CI/CD (changes to configs of workflows)
  • 💥 BREAKING CHANGE (fix or feature that require a new minimal version of the front-end)
  • 😶‍🌫️ No impact for the end-users

Impact & Scope

  • Core functionality changes
  • Single module changes
  • Multiple modules changes
  • Database migrations required
  • Other: ...

Testing

  • 1. Tested this locally
  • 2. Added/modified tests that pass the CI (or tested in a downstream fork)
  • 3. Tested in a deployed pre-prod
  • 0. Untestable (exceptionally), will be tested in prod directly

Documentation

  • Updated the docs accordingly :
  • " Docstrings
  • # Inline comments
  • No documentation needed

@armanddidierjean armanddidierjean changed the title Use async files manipulation using anyio Async files manipulation using anyio Apr 12, 2026
@codecov

codecov Bot commented Apr 12, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.30137% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.43%. Comparing base (33f63b5) to head (0c17664).

Files with missing lines Patch % Lines
app/modules/raid/endpoints_raid.py 50.00% 2 Missing ⚠️
...s/sport_competition/endpoints_sport_competition.py 50.00% 2 Missing ⚠️
app/utils/tools.py 91.30% 2 Missing ⚠️
app/core/mypayment/endpoints_mypayment.py 50.00% 1 Missing ⚠️
app/modules/calendar/endpoints_calendar.py 66.66% 1 Missing ⚠️
app/modules/cinema/endpoints_cinema.py 0.00% 1 Missing ⚠️
app/modules/phonebook/endpoints_phonebook.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #985      +/-   ##
==========================================
- Coverage   84.43%   84.43%   -0.01%     
==========================================
  Files         206      206              
  Lines       14952    14956       +4     
==========================================
+ Hits        12625    12628       +3     
- Misses       2327     2328       +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@armanddidierjean armanddidierjean marked this pull request as ready for review April 12, 2026 10:02
@armanddidierjean armanddidierjean mentioned this pull request Apr 12, 2026
19 tasks
@warix8

warix8 commented Apr 12, 2026

Copy link
Copy Markdown
Contributor

Could you please update https://docs.myecl.fr/hyperion/other/data_files.html accordingly ?

@armanddidierjean

Copy link
Copy Markdown
Member Author

Could you please update https://docs.myecl.fr/hyperion/other/data_files.html accordingly ?

See aeecleclair/myecl-documentation#15

Comment thread app/core/utils/config.py
@NakoGH NakoGH merged commit c48f6b0 into main Jun 5, 2026
7 checks passed
@NakoGH NakoGH deleted the anyio branch June 5, 2026 23:19
Toshyb13 pushed a commit that referenced this pull request Jun 16, 2026
# Description

## Summary

<!--BRIEF description: DON'T EXPLAIN the code: JUSTIFY what this PR is
for!-->

Ruff 0.15.10 compatibility

### Required PRs

<!--Keywords: "depends on", "blocked by" -->
<!--Depends on #-->

Requires #985

<!--Don't touch thses two tags-->
<details>
<summary>

# Classification

</summary>

## Type of Change

- [ ] 🐛 Bug fix (non-breaking change which fixes an issue)
- [ ] ✨ New feature (non-breaking change which adds functionality)
- [x] 🔨 Refactor (non-breaking change that neither fixes a bug nor adds
a feature)
- [ ] 🔧 Infra CI/CD (changes to configs of workflows)
- [ ] 💥 BREAKING CHANGE (fix or feature that require a new minimal
version of the front-end)
- [x] 😶‍🌫️ No impact for the end-users

## Impact & Scope

- [ ] Core functionality changes
- [ ] Single module changes
- [ ] Multiple modules changes
- [ ] Database migrations required
- [ ] Other: ... <!--Not module-oriented: write something!-->

## Testing

- [ ] 1. Tested this locally
- [ ] 2. Added/modified tests that pass the CI (or tested in a
downstream fork)
- [ ] 3. Tested in a deployed pre-prod
- [ ] 0. Untestable (exceptionally), will be tested in prod directly

## Documentation

- [ ] Updated [the docs](docs.myecl.fr) accordingly : <!--[Docs#0 -
Title](https://github.com/aeecleclair/myecl-documentation/pull/0)-->
- [ ] `"` Docstrings
- [ ] `#` Inline comments
- [ ] No documentation needed

</details>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants