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

feat: update to spago@next #20

Merged
merged 4 commits into from
Apr 2, 2024
Merged

Conversation

srghma
Copy link
Member

@srghma srghma commented Mar 12, 2024

Description of the change

update to spago@next

locally tests dont work, but on ci - work


Checklist:

  • Added the change to the changelog's "Unreleased" section with a link to this PR and your username
  • Linked any existing issues or proposals that this pull request should close
  • Updated or added relevant documentation in the README and/or documentation directory
  • Added a test for the contribution (if applicable)

locally tests dont work, but on ci - work
packages.dhall_ Outdated Show resolved Hide resolved
@natefaubion
Copy link

I believe latest spago has landed on camelCase fields instead snake_case, FWIW.

@srghma
Copy link
Member Author

srghma commented Mar 18, 2024

I believe latest spago has landed on camelCase fields instead snake_case, FWIW.

again update everything

sd -s "packageSet" "package_set" $(fd --type file -e yaml -e yml -e purs)
sd -s "extraPackages" "extra_packages" $(fd --type file -e yaml -e yml -e purs)
# or 
spago build --migrate ?

NB: not yet published to npm

@thomashoneyman
Copy link
Collaborator

Looks like the spago in CI is old? And — we don’t need psa anymore if we use the new spago, it’s bundled in.

@thomashoneyman thomashoneyman merged commit df3461b into purescript-contrib:main Apr 2, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants