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.

Sign up for GitHub

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

Jump to bottom

docs(readme): update min. requirements + cleanup #10750

Merged
merged 3 commits into from
Mar 14, 2025
Merged

docs(readme): update min. requirements + cleanup #10750

merged 3 commits into from
Mar 14, 2025

Conversation

lidel
Copy link
Member

@lidel lidel commented Mar 11, 2025 •
edited
Loading

  • Adding note about extra memory requirement due to ba22102
  • Moved min requirements into "Install" section so it is hard to miss
  • removed outdated info

cc @2color @hsanjuan for proof-reading 🙏

@lidel
docs(readme): update min. requirements + cleanup
7225889 
Adding note about extra memory requirement due to
ba22102
@lidel lidel requested a review from 2color March 11, 2025 13:39
@lidel lidel added the skip/changelog This change does NOT require a changelog entry label Mar 11, 2025
@lidel lidel marked this pull request as ready for review March 11, 2025 13:51
@lidel lidel requested a review from a team as a code owner March 11, 2025 13:51
@lidel lidel mentioned this pull request Mar 11, 2025
Closed
56 tasks
2color
2color reviewed Mar 11, 2025
View reviewed changes
@lidel @2color
chore: update docs/config.md
1505a44 
Co-authored-by: Daniel Norman <1992255+2color@users.noreply.github.com>
@lidel lidel merged commit 96f3007 into master Mar 14, 2025
2 checks passed
@lidel lidel deleted the docs-min-reqs-cleanup branch March 14, 2025 16:55
lidel added a commit that referenced this pull request Mar 14, 2025
@lidel
docs(readme): update min. requirements + cleanup (#10750)
f994d94 
* docs(readme): update min. requirements + cleanup

Adding note about extra memory requirement due to
ba22102

* docs(config): memory cost of buffered provider

Co-authored-by: Daniel Norman <1992255+2color@users.noreply.github.com>
(cherry picked from commit 96f3007)
@BrewTestBot BrewTestBot mentioned this pull request Mar 20, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@2color 2color 2color left review comments

@hsanjuan hsanjuan hsanjuan approved these changes

Assignees
No one assigned
Labels
skip/changelog This change does NOT require a changelog entry
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lidel @hsanjuan @2color