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 for 3.5.x, deals with iotesters 3.5.x refactor #222

Closed
wants to merge 3 commits into from

Conversation

chick
Copy link
Contributor

@chick chick commented Feb 3, 2021

Depends on freechipsproject/chisel-testers#309
Depends on chipsalliance/chisel#1730

Black boxes required suggest Name to work
setResource on black box is not addResource
Lots of import cleanup
Saturate did it's own firrtl compiling, modified to use current idioms
Changed dsptools Driver to use jiuyangs iotesters DriverCompatibility stuff
Cleaned up dependencies in build.sbt
Removed BuildInfo plugin from build.sbt (I'm not that sure about it, but it was causing build errors)

setResource on black box is not addResource
Lots of import cleanup
Saturate did it's own firrtl compiling, modified to use current idioms
Changed dsptools `Driver` to use jiuyangs iotesters DriverCompatibility stuff
Cleaned up dependencies in build.sbt
Removed BuildInfo plugin from build.sbt (I'm not that sure about it, but it was causing build errors)
@chick chick self-assigned this Feb 3, 2021
@chick chick added this to the 1.5.x milestone Feb 3, 2021
@chick
Copy link
Contributor Author

chick commented Feb 3, 2021

Will not build until above-mentioned PRs go through

chick added a commit that referenced this pull request Apr 19, 2021
This compbined PR #222 AND PR #222

Depends on freechipsproject/chisel-testers#309
Depends on chipsalliance/chisel#1730

Black boxes required suggest Name to work
setResource on black box is not addResource
Lots of import cleanup
Saturate did it's own firrtl compiling, modified to use current idioms
Changed dsptools Driver to use jiuyangs iotesters DriverCompatibility stuff
Cleaned up dependencies in build.sbt
Removed BuildInfo plugin from build.sbt (I'm not that sure about it, but it was causing build errors)
--
Now uses github actions for CI
chick added a commit that referenced this pull request Apr 19, 2021
* Black boxes required suggest Name to work
setResource on black box is not addResource
Lots of import cleanup
Saturate did it's own firrtl compiling, modified to use current idioms
Changed dsptools `Driver` to use jiuyangs iotesters DriverCompatibility stuff
Cleaned up dependencies in build.sbt
Removed BuildInfo plugin from build.sbt (I'm not that sure about it, but it was causing build errors)

* Add support for github actions to do ci and publishing

* Add support for github actions to do ci and publishing
Added CODEOWNERS file for symmetry with chiseltest which was used for this template
Removed mill as it is not currently supported here

* Update plugins to have ci-release, used for publishing

* fix iotesters version

* bump copyright to force retest

* Move to github actions for ci, update for 3.5 compatabilty
This compbined PR #222 AND PR #222

Depends on freechipsproject/chisel-testers#309
Depends on chipsalliance/chisel#1730

Black boxes required suggest Name to work
setResource on black box is not addResource
Lots of import cleanup
Saturate did it's own firrtl compiling, modified to use current idioms
Changed dsptools Driver to use jiuyangs iotesters DriverCompatibility stuff
Cleaned up dependencies in build.sbt
Removed BuildInfo plugin from build.sbt (I'm not that sure about it, but it was causing build errors)
--
Now uses github actions for CI

* Only build for scala 2.12
@chick
Copy link
Contributor Author

chick commented Apr 19, 2021

Closed. Pr #227 has replaced this PR

@chick chick closed this Apr 19, 2021
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.

2 participants