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

Onchain code for programmable tokens #4

Merged
merged 15 commits into from
Dec 18, 2024
Merged

Onchain code for programmable tokens #4

merged 15 commits into from
Dec 18, 2024

Conversation

colll78
Copy link
Collaborator

@colll78 colll78 commented Dec 16, 2024

Introduce the onchain code for the wst-poc.

Copy link
Collaborator

@j-mueller j-mueller left a comment

Choose a reason for hiding this comment

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

There are two files under src/lib/SmartTokens/LinkedList that are completely unused, do we need them? One of them even has a different module name than the file name suggests. Ie. AirdropEvent.Mint.Common. Maybe we should move this stuff to the standard library?

j-mueller and others added 4 commits December 17, 2024 10:01
* Otherwise ghc and friends will be compiled from source.

* There is a nix failure independent of this change which can be fixed
  by deleting the .pre-commit-config.yaml. This should be checked.
* This file should be auto-generated and its existence prevents nix
  flake development environments being created.
@j-mueller
Copy link
Collaborator

FYI I had to delete the entry for nixpkgs-fmt from the auto-generated pre-commit-config.yaml. It failed with an index out of range error, making it impossible to commit anything.

colll78 and others added 6 commits December 17, 2024 16:38
* Stylish Haskell configuration included

* Disabled nixpkgs-fmt (currently no idea why it breaks in the hook)

* nonix pre-commit-hook (not tested myself)
@j-mueller j-mueller merged commit 992309b into main Dec 18, 2024
2 checks passed
@j-mueller j-mueller deleted the onchain-new branch December 18, 2024 08:41
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.

3 participants