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

SRE-36 haskell overhaul #1194

Merged
merged 1 commit into from Feb 14, 2020
Merged

SRE-36 haskell overhaul #1194

merged 1 commit into from Feb 14, 2020

Conversation

craigem
Copy link
Contributor

@craigem craigem commented Feb 6, 2020

This is the overhaul of Haskell for cardano-ledger-specs along the lines of cardano-node PR #507.

@craigem craigem requested a review from nc6 as a code owner February 6, 2020 07:49
nc6
nc6 previously requested changes Feb 10, 2020
Copy link
Contributor

@nc6 nc6 left a comment

Choose a reason for hiding this comment

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

Per inline comments, please either remove commented lines or reinstate them

.buildkite/pipeline.yml Outdated Show resolved Hide resolved
cabal.project Show resolved Hide resolved
@jbgi jbgi force-pushed the SRE-36_haskell_overhaul branch 6 times, most recently from bb90663 to 9bf0d78 Compare February 12, 2020 18:36
@disassembler disassembler force-pushed the SRE-36_haskell_overhaul branch 2 times, most recently from 3ecf06f to 3c5283e Compare February 13, 2020 06:38
@jbgi jbgi force-pushed the SRE-36_haskell_overhaul branch 2 times, most recently from 13545cf to 3d6f81b Compare February 13, 2020 18:15
@disassembler
Copy link
Contributor

bors r+

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Feb 13, 2020

👎 Rejected by code reviews

Copy link
Contributor

@dcoutts dcoutts left a comment

Choose a reason for hiding this comment

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

I'm satisfied that the review comments have been addressed.

@dcoutts dcoutts dismissed nc6’s stale review February 13, 2020 19:01

I'm satisfied that the review comments have now been addressed.

@disassembler
Copy link
Contributor

bors r+

iohk-bors bot added a commit that referenced this pull request Feb 13, 2020
1194: SRE-36 haskell overhaul  r=disassembler a=craigem

This is the overhaul of Haskell for `cardano-ledger-specs` along the lines of `cardano-node` [PR #507](IntersectMBO/cardano-node#507).

Co-authored-by: Craige McWhirter <craige.mcwhirter@iohk.io>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Feb 13, 2020

Build failed

@jbgi
Copy link
Contributor

jbgi commented Feb 13, 2020

bors retry

iohk-bors bot added a commit that referenced this pull request Feb 13, 2020
1194: SRE-36 haskell overhaul  r=disassembler a=craigem

This is the overhaul of Haskell for `cardano-ledger-specs` along the lines of `cardano-node` [PR #507](IntersectMBO/cardano-node#507).

Co-authored-by: Craige McWhirter <craige.mcwhirter@iohk.io>
@disassembler
Copy link
Contributor

bors r-

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Feb 13, 2020

Canceled

@disassembler
Copy link
Contributor

bors r+

iohk-bors bot added a commit that referenced this pull request Feb 13, 2020
1194: SRE-36 haskell overhaul  r=disassembler a=craigem

This is the overhaul of Haskell for `cardano-ledger-specs` along the lines of `cardano-node` [PR #507](IntersectMBO/cardano-node#507).

Co-authored-by: Craige McWhirter <craige.mcwhirter@iohk.io>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Feb 13, 2020

Build failed

@disassembler
Copy link
Contributor

bors r+

@disassembler
Copy link
Contributor

bors r-

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Feb 14, 2020

Canceled

@disassembler
Copy link
Contributor

bors r+

iohk-bors bot added a commit that referenced this pull request Feb 14, 2020
1194: SRE-36 haskell overhaul  r=disassembler a=craigem

This is the overhaul of Haskell for `cardano-ledger-specs` along the lines of `cardano-node` [PR #507](IntersectMBO/cardano-node#507).

Co-authored-by: Craige McWhirter <craige.mcwhirter@iohk.io>
@disassembler
Copy link
Contributor

bors r-

@disassembler
Copy link
Contributor

bors r+

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Feb 14, 2020

Canceled

iohk-bors bot added a commit that referenced this pull request Feb 14, 2020
1194: SRE-36 haskell overhaul  r=disassembler a=craigem

This is the overhaul of Haskell for `cardano-ledger-specs` along the lines of `cardano-node` [PR #507](IntersectMBO/cardano-node#507).

Co-authored-by: Craige McWhirter <craige.mcwhirter@iohk.io>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Feb 14, 2020

@iohk-bors iohk-bors bot merged commit ca2cdd2 into master Feb 14, 2020
@iohk-bors iohk-bors bot deleted the SRE-36_haskell_overhaul branch February 14, 2020 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants