Skip to content

Migrate from spago-legacy to spago - #18

Merged
f-f merged 10 commits into
sigma-andex:mainfrom
m-rinaldi:spago-migration
Jun 24, 2026
Merged

Migrate from spago-legacy to spago#18
f-f merged 10 commits into
sigma-andex:mainfrom
m-rinaldi:spago-migration

Conversation

@m-rinaldi

Copy link
Copy Markdown
Contributor

migrate to new spago (spago.yaml)

Summary

Migrates the build tooling from spago-legacy (Dhall-based) to the current spago (YAML-based).

Changes

spago.yaml — test configuration

The old test.dhall extended spago.dhall with extra source globs and test-only dependencies. In new spago, test configuration lives directly under package.test in spago.yaml. However, new spago's package.test does not support custom sources — only test/**/*.purs is used for test sources.

docs/Examples/test/Examples/

The example .purs and .js files were previously included in the test build via the docs/Examples/**/*.purs glob in test.dhall. Since that glob can no longer be expressed in spago.yaml, and the examples exist primarily as test fixtures (imported directly by IntegrationSpec.purs), they have been moved to test/Examples/. Module names and all imports are unchanged. Documentation (Readme.md, assets) remains in docs/Examples/.

test.dhall removed

No longer needed; superseded by the package.test section in spago.yaml.

Compiler warnings fixed

Removed unused imports across example files and TestHelpers.purs, and fixed warnings in RequestSpec.purs.

Comment thread spago.yaml Outdated
Comment thread test/Test/Main.purs Outdated
@f-f

f-f commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

@m-rinaldi could you also upgrade CI while we are at it? it's using deprecated actions

@m-rinaldi
m-rinaldi requested a review from f-f June 15, 2026 18:22
@sigma-andex

Copy link
Copy Markdown
Owner

Seems spago is still on an old version

@f-f

f-f commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

yeah I think this will need updating

@m-rinaldi

Copy link
Copy Markdown
Contributor Author

On the way: purescript-contrib/setup-purescript#47

@f-f f-f left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @m-rinaldi this is great!!

@f-f
f-f merged commit 2d0a926 into sigma-andex:main Jun 24, 2026
1 check passed
@m-rinaldi
m-rinaldi deleted the spago-migration branch June 24, 2026 17:01
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