Skip to content

Updates for PureScript 0.15#120

Merged
fsoikin merged 11 commits intomasterfrom
trh/purs-0.15
Apr 29, 2022
Merged

Updates for PureScript 0.15#120
fsoikin merged 11 commits intomasterfrom
trh/purs-0.15

Conversation

@thomashoneyman
Copy link
Copy Markdown
Collaborator

This makes a number of updates so the library will be compatible with all the other PureScript 0.15 library releases.


build-example: $(OUTPUT)
pulp build -I example --to $(OUTPUT)/example.js
npx spago bundle-app --path $(example) --to $(OUTPUT)/example.js --minify
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Isn't bundling functionality going away?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Native bundling is going away, and instead Spago will call out to esbuild under the hood (if it's installed), or you can swap in another bundler.

@thomashoneyman
Copy link
Copy Markdown
Collaborator Author

@fsoikin I seem to have busted something related to the ansi dependency, but it's quite late and I don't have any more time today. I'll come back tomorrow to see about getting this building!

@thomashoneyman
Copy link
Copy Markdown
Collaborator Author

This should be good to go pending felixSchl/purescript-pipes#21

@thomashoneyman
Copy link
Copy Markdown
Collaborator Author

@fsoikin this is now ready for review, if you'd like!

Copy link
Copy Markdown
Member

@fsoikin fsoikin 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!

@fsoikin fsoikin merged commit e2f0f1a into master Apr 29, 2022
@fsoikin fsoikin deleted the trh/purs-0.15 branch April 29, 2022 22:17
@fsoikin
Copy link
Copy Markdown
Member

fsoikin commented Apr 30, 2022

Cut and published v7.0.0, thanks @thomashoneyman!

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.

2 participants