Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

Simplify Github Actions CI #267

Merged
merged 1 commit into from
Nov 18, 2020
Merged

Simplify Github Actions CI #267

merged 1 commit into from
Nov 18, 2020

Conversation

jackkoenig
Copy link
Collaborator

WIP

@jackkoenig jackkoenig requested a review from a team as a code owner November 18, 2020 22:48
Copy link
Collaborator

@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.

That's a very nice simplification

@jackkoenig
Copy link
Collaborator Author

A lot of the code before was dealing with using a Docker image for dependencies like Verilator; none of which are needed by this repo. I suspect we'll want to refer back to it if we were to move Chisel or FIRRTL to Github Actions.

That being said, we might as well make it as simple as possible for Treadle since we have that luxury 🙂

@jackkoenig
Copy link
Collaborator Author

Also some of it was for building dependencies which becomes unnecessary with the auto SNAPSHOT publishing

@jackkoenig
Copy link
Collaborator Author

@Mergifyio backport 1.3.x 1.2.x

mergify bot pushed a commit that referenced this pull request Nov 19, 2020
(cherry picked from commit 7e46a00)
@mergify
Copy link
Contributor

mergify bot commented Nov 19, 2020

Command backport 1.3.x 1.2.x: failure

No backport have been created

jackkoenig added a commit that referenced this pull request Nov 19, 2020
(cherry picked from commit 7e46a00)

Co-authored-by: Jack Koenig <koenig@sifive.com>
jackkoenig added a commit that referenced this pull request Nov 19, 2020
(cherry picked from commit 7e46a00)
jackkoenig added a commit that referenced this pull request Nov 19, 2020
@jackkoenig jackkoenig added this to the 1.2.x milestone Nov 24, 2020
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants