Skip to content

Conversation

seldridge
Copy link
Member

Change CI to build Treadle master from source. This fixes a bug where
Treadle would be pulled from the latest snapshot (which would pull the
latest FIRRTL snapshot). For bleeding edge PRs that rely on new FIRRTL
features, using a FIRRTL snapshot may result in compilation failures.

Signed-off-by: Schuyler Eldridge schuyler.eldridge@ibm.com

Related issue: #1499 (comment)

Type of change: other enhancement

Impact: no functional change

Development Phase: implementation

Release Notes

None.

Change CI to build Treadle master from source. This fixes a bug where
Treadle would be pulled from the latest snapshot (which would pull the
latest FIRRTL snapshot). For bleeding edge PRs that rely on new FIRRTL
features, using a FIRRTL snapshot may result in compilation failures.

Signed-off-by: Schuyler Eldridge <schuyler.eldridge@ibm.com>
@seldridge seldridge requested a review from a team as a code owner July 15, 2020 19:58
@seldridge seldridge requested a review from chick July 15, 2020 20:27
@seldridge
Copy link
Member Author

I added one commit, marked with !remove, which adds a reference to the new firrtl.ir.Formal package to show that this PR fixes the build issues mentioned in #1499 (comment). Pending approval, I'll strip this commit via rebase, but am leaving it in for a reviewer to verify that this does do what it's supposed to do.

Copy link
Contributor

@chick chick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@seldridge seldridge added this to the 3.4.0 milestone Jul 15, 2020
@seldridge seldridge added the Please Merge Accepted PRs that are ready to be merged. Useful when waiting on CI. label Jul 15, 2020
@mergify mergify bot merged commit 4a0e828 into master Jul 15, 2020
@seldridge seldridge deleted the ci-treadle branch July 27, 2020 17:12
jackkoenig pushed a commit that referenced this pull request Feb 28, 2023
* Fixes #1516
* Tighten up logic for "casted literal" checking
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Please Merge Accepted PRs that are ready to be merged. Useful when waiting on CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants