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

refactor: Rename Component.async_run to Component.run_async for better readablility #8370

Merged
merged 1 commit into from
Sep 17, 2024

Conversation

shadeMe
Copy link
Contributor

@shadeMe shadeMe commented Sep 16, 2024

Related Issues

Proposed Changes:

Using a suffix will keep names logically sorted, less noisy and relegate the async aspect to an implementation/API detail.

How did you test it?

CI

Checklist

@shadeMe shadeMe requested review from a team as code owners September 16, 2024 14:25
@shadeMe shadeMe requested review from dfokina and silvanocerza and removed request for a team September 16, 2024 14:25
@coveralls
Copy link
Collaborator

coveralls commented Sep 16, 2024

Pull Request Test Coverage Report for Build 10900657906

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 90.27%

Totals Coverage Status
Change from base Build 10899485113: 0.0%
Covered Lines: 7162
Relevant Lines: 7934

💛 - Coveralls

…etter readablility

Using a suffix will keep names logically sorted, less noisy and relegate the async aspect to an implementation/API detail.
@shadeMe shadeMe force-pushed the refactor/rename-async-run branch from 89db502 to e342da4 Compare September 17, 2024 09:51
@shadeMe shadeMe added the ignore-for-release-notes PRs with this flag won't be included in the release notes. label Sep 17, 2024
@github-actions github-actions bot added the type:documentation Improvements on the docs label Sep 17, 2024
@shadeMe shadeMe enabled auto-merge (squash) September 17, 2024 10:02
@shadeMe shadeMe merged commit 5071e47 into deepset-ai:main Sep 17, 2024
19 checks passed
@shadeMe shadeMe deleted the refactor/rename-async-run branch September 17, 2024 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ignore-for-release-notes PRs with this flag won't be included in the release notes. topic:core topic:tests type:documentation Improvements on the docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants