Skip to content

Conversation

@wanda-phi
Copy link
Member

Fixes #875.

Note: this depends on #840 and will fail tests without it.

@whitequark whitequark added this to the 0.5 milestone Nov 2, 2023
@whitequark whitequark changed the title Implement RFC 20: Remove non-FWFT FIFOs. (Merge after 0.4 is released) Remove non-FWFT FIFOs Nov 25, 2023
@whitequark
Copy link
Member

Note: this will need another PR to add a changelog entry after merging.

@wanda-phi wanda-phi marked this pull request as ready for review December 13, 2023 11:32
@codecov
Copy link

codecov bot commented Dec 13, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9d4ffab) 86.62% compared to head (e81554e) 86.58%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #949      +/-   ##
==========================================
- Coverage   86.62%   86.58%   -0.05%     
==========================================
  Files          41       41              
  Lines        7043     7035       -8     
  Branches     1678     1675       -3     
==========================================
- Hits         6101     6091      -10     
- Misses        779      782       +3     
+ Partials      163      162       -1     

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

@whitequark
Copy link
Member

@wanda-phi Changelog?

@whitequark
Copy link
Member

Thanks!

@whitequark whitequark added this pull request to the merge queue Dec 13, 2023
Merged via the queue into amaranth-lang:main with commit 8cd8cdd Dec 13, 2023
@wanda-phi wanda-phi deleted the remove-fwft branch December 13, 2023 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Implement RFC 20: Deprecate and remove non-FWFT FIFOs

2 participants