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

Use version of canvas-action from package set #2

Merged
merged 2 commits into from
Nov 18, 2021

Conversation

maxdeviant
Copy link
Contributor

This PR makes it so grid-reactors depends on the version of canvas-action from the package set instead of an arbitrary Git tag.

As part of this, we also had to upgrade to a newer version of the package set.

Motivation

grid-reactors is currently in the PureScript package set and will need to have these dependency issues resolved in order to remain in the package set after the move to the new PureScript registry.

See purescript/registry#250 for more details.

@@ -1,37 +1,45 @@
{
"name": "purescript-grid-reactors",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The churn in this file is due to spago bump-version replacing the bower.json.


in upstream
with canvas-action =
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We no longer need this override as we can use the version of canvas-action from the package set.

The only difference between the version in the package set and the version that was previously referenced is a README change: artemisSystem/purescript-canvas-action@v7.0.0...43de19e

@Eugleo
Copy link
Owner

Eugleo commented Nov 18, 2021

Thank you, much appreciated!

@maxdeviant
Copy link
Contributor Author

Thank you, much appreciated!

No problem!

Would you mind publishing a new version so I can update this in the package set?

@Eugleo
Copy link
Owner

Eugleo commented Nov 18, 2021

No at all. First, I'll need to remind myself how to do that ;-)

I'll also update the npm deps to fix some (perceived) vulnerabilities when I'm at it.

@Eugleo
Copy link
Owner

Eugleo commented Nov 18, 2021

By the way, is this post on Pursuit a good checklist to follow when publishing packages?

@maxdeviant
Copy link
Contributor Author

By the way, is this post on Pursuit a good checklist to follow when publishing packages?

While I think that page has all the necessary information, I think this Discourse post might be a bit easier to follow.

@Eugleo
Copy link
Owner

Eugleo commented Nov 18, 2021

Thank you, good find. Anyway, everything's done. Shall I update the package set, or will you do it?

@maxdeviant
Copy link
Contributor Author

Thank you, good find. Anyway, everything's done. Shall I update the package set, or will you do it?

I'll be able to take care of it, unless you're really dying to do it 😄

@maxdeviant
Copy link
Contributor Author

Thank you, good find. Anyway, everything's done. Shall I update the package set, or will you do it?

I'll be able to take care of it, unless you're really dying to do it 😄

@Eugleo Would you mind pushing the v3.0.0 tag to this repository? I'm not seeing it at the moment.

@Eugleo
Copy link
Owner

Eugleo commented Nov 18, 2021

Not really, I'll gladly let you do the honors :-D You'll be adding it to the next coming package set, I assume? Meaning, if I want to use the newest version now, I'll have to refer to it by the commit hash for the time being, correct?

@maxdeviant
Copy link
Contributor Author

Not really, I'll gladly let you do the honors. You'll be adding it to the next coming package set, I assume? Meaning, if I want to use the newest version now, I'll have to refer to it by the commit hash for the time being, correct?

Correct, it would be in the next version of the package set (although we're usually pretty good about putting those out regularly; shouldn't be more than a couple days).

Until then you would have to add it as an addition to your packages.dhall file if you wanted to use the latest version.

@Eugleo
Copy link
Owner

Eugleo commented Nov 18, 2021

Ok. I just pushed the tag.

@maxdeviant
Copy link
Contributor Author

Ok. I just pushed the tag.

Thank you!

Here's the PR to add the new version to the package set: purescript/package-sets#1033

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