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

Deprecating cocotb.generators and Scoreboard #2047

Merged
merged 4 commits into from
Sep 5, 2020

Conversation

ktbarrett
Copy link
Member

This rebases #2041 on master. It deprecates everything in the cocotb.generators package, and the cocotb.scoreboard.Scoreboard. It must also update the tests and examples that use the deprecated feature.

We must use conda on Windows to build the cocotb package for CI. We
shouldn't have Travis source a python installation because we aren't
using it.
@ktbarrett ktbarrett added the type:deprecation API that should warn and eventually be removed label Sep 5, 2020
@codecov
Copy link

codecov bot commented Sep 5, 2020

Codecov Report

Merging #2047 into master will increase coverage by 0.03%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2047      +/-   ##
==========================================
+ Coverage   64.20%   64.23%   +0.03%     
==========================================
  Files          34       34              
  Lines        4724     4728       +4     
  Branches      816      816              
==========================================
+ Hits         3033     3037       +4     
  Misses       1380     1380              
  Partials      311      311              
Impacted Files Coverage Δ
cocotb/generators/bit.py 80.00% <ø> (ø)
cocotb/generators/packet.py 0.00% <ø> (ø)
cocotb/generators/__init__.py 81.48% <100.00%> (+1.48%) ⬆️
cocotb/generators/byte.py 72.72% <100.00%> (ø)
cocotb/scoreboard.py 33.96% <100.00%> (+1.26%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 78126f0...6b33496. Read the comment docs.

ktbarrett added a commit to ktbarrett/cocotb that referenced this pull request Sep 5, 2020
@ktbarrett ktbarrett merged commit 0d03064 into cocotb:master Sep 5, 2020
@ktbarrett ktbarrett deleted the fix-scoreboard branch September 5, 2020 23:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:deprecation API that should warn and eventually be removed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants