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

PlutusTx as a separate backend #227

Merged
merged 24 commits into from
Jun 24, 2024
Merged

Conversation

bladyjoker
Copy link
Contributor

@bladyjoker bladyjoker commented Jun 12, 2024

Tackles #197 #221

DONE

  • Misc:
    • Refactored Codegen and make the relationship between different components clearer
    • Added extras/dev-shells where the dev-prelude|plutus-haskell shells are
  • Separates PlutusTx:
    • Its own lbf-plutus-to-plutustx CLI
    • Its own lbfPlutusTx Nix function
    • Its own lbr-plutustx
    • Complete test suite that RUNS the compiled scripts
  • Plutarch testsuite
    • Complete test suite
  • LB Plutus
    • Removed Plutus.V1.Todo and PlutusV2.Todo
    • Did some codegen config cleanup and aligned them all a bit better (there were duplicates and missing stuff)
  • Purescript
    • Made a LambdaBuffers.Runtime.Plutus.NotImplemented type and mapped all currently not implemented Opaques there

TODO

@bladyjoker bladyjoker added bug Something isn't working enhancement New feature or request codegen labels Jun 12, 2024
@bladyjoker bladyjoker self-assigned this Jun 12, 2024
@bladyjoker bladyjoker changed the title Separates PlutusTx as a separated backend PlutusTx as a separate backend Jun 12, 2024
Copy link
Collaborator

@szg251 szg251 left a comment

Choose a reason for hiding this comment

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

I tried my best to read through all the code, but this thing is huge 😆
I couldn't spot anything, I liked the refactorings.

"LambdaBuffers.Runtime.Plutus",
"TxInInfo"
"NotImplemented"
Copy link
Collaborator

Choose a reason for hiding this comment

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

meh 😒

@@ -0,0 +1 @@
use flake ../../..#dev-lbr-plutustx
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
use flake ../../..#dev-lbr-plutustx
use flake .#dev-lbr-plutustx

@szg251 szg251 merged commit 7715618 into szg251/tx-info Jun 24, 2024
2 checks passed
@szg251 szg251 deleted the bladyjoker/separate-plutustx branch June 24, 2024 14:25
chfanghr added a commit to mlabs-haskell/lambda-buffers-for-cardano that referenced this pull request Jun 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working codegen enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants