Skip to content

package-set-0.12 #171

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

Merged
merged 7 commits into from
Apr 30, 2018
Merged

package-set-0.12 #171

merged 7 commits into from
Apr 30, 2018

Conversation

chexxor
Copy link
Contributor

@chexxor chexxor commented Apr 28, 2018

This adds the SHAs for the core packages (those in the purescript org), from this list: purescript/purescript#3116

I need to figure out how to verify this package.json file - it looks like psc-package verify expects a psc-package.json file...

Until then, though, I welcome reviews.

@chexxor
Copy link
Contributor Author

chexxor commented Apr 28, 2018

Will probably need to remove all non-core packages.

@justinwoo
Copy link
Collaborator

justinwoo commented Apr 28, 2018

Should add this as a branch first? like 0.12-rc or something

edit: I don't know if GitHub UI actually allows for PRs that create a branch 😄

@LiamGoodacre
Copy link
Member

Yeah you'll need to create a new branch in this repo and then update this PR to switch base branch.

@justinwoo
Copy link
Collaborator

@LiamGoodacre LiamGoodacre changed the base branch from master to 0.12-rc April 28, 2018 23:12
@LiamGoodacre
Copy link
Member

Will eventually need to update travis.yml to point at the 0.12-rc compiler release once that is available.

@justinwoo
Copy link
Collaborator

Could you reset the commit and force push it? Travis seems to not understand how to use the correct commit.

packages.json Outdated
@@ -1257,21 +1235,12 @@
"repo": "https://github.com/Thimoteus/purescript-mmorph.git",
"version": "v3.0.0"
},
"monoid": {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Monoid is missing?

Copy link
Member

Choose a reason for hiding this comment

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

Monoid is deprecated.

Copy link
Member

Choose a reason for hiding this comment

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

The type class has moved to Prelude.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ah, then all the listings of dependencies should be removed. Cool

Copy link
Member

Choose a reason for hiding this comment

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

@justinwoo
Copy link
Collaborator

In addition, reverse dependencies of typelevel-prelude won't work without upgrades themselves, so they need to be removed. E.g. Simple-JSON

@chexxor
Copy link
Contributor Author

chexxor commented Apr 29, 2018

Yeah, I'll work on that.

@chexxor
Copy link
Contributor Author

chexxor commented Apr 29, 2018

I just removed all non-core libs. Can re-add other libs as they get upgraded to work with new core libs.

@chexxor
Copy link
Contributor Author

chexxor commented Apr 29, 2018

Requires some PRs be merged in some libs:
purescript/purescript-exceptions#28
purescript-deprecated/purescript-inject#10 (also require some edits before merging)

@chexxor
Copy link
Contributor Author

chexxor commented Apr 29, 2018

Removed "inject" package, as that'll be merged into functors/either. purescript-deprecated/purescript-inject#10

@justinwoo
Copy link
Collaborator

Exceptions needs an update or a new target?

@justinwoo
Copy link
Collaborator

justinwoo commented Apr 30, 2018

Oh, builds locally just fine... edit: oh, it's probably that the Travis cache messes it up.

Builds just fine without cache, so merging this into the branch. https://travis-ci.org/justinwoo/package-sets/builds/373085891

Thanks!!

@chexxor
Copy link
Contributor Author

chexxor commented Apr 30, 2018 via email

@justinwoo justinwoo merged commit 949e3eb into purescript:0.12-rc Apr 30, 2018
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