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

New package: volta-0.7.1 #18756

Closed
wants to merge 1 commit into from
Closed

New package: volta-0.7.1 #18756

wants to merge 1 commit into from

Conversation

atk
Copy link
Contributor

@atk atk commented Feb 3, 2020

See #18446. Unfortunately, I was not able to reopen the PR due to the force push (my bad). The comments from this PR have been resolved.

@atk atk force-pushed the feat-volta branch 2 times, most recently from 84ed616 to bc8f8b2 Compare February 3, 2020 11:32
@atk atk requested a review from xtraeme February 3, 2020 11:50
srcpkgs/volta/template Outdated Show resolved Hide resolved
@Vaelatern
Copy link
Member

@atk you are able to re-open the PR, you just need to force push to that branch. I'm pretty sure it can be done.

@Vaelatern
Copy link
Member

Still an installation script. I still do not like this.

@atk
Copy link
Contributor Author

atk commented Feb 3, 2020

I just had a chat with the volta developers. The scripts are no longer required, so both those and the install script can go.

@atk
Copy link
Contributor Author

atk commented Feb 3, 2020

I've checked back, there is no musl support for volta - yet. However, since the demand is there for something like volta to be used inside containers (for which Alpine LInux is very popular), it might be supported in the future. See: volta-cli/volta#473

@atk
Copy link
Contributor Author

atk commented Feb 3, 2020

Removed the superfluous patch.

@atk
Copy link
Contributor Author

atk commented Feb 4, 2020

@Vaelatern, the button says I cannot reopen the PR because I force-pushed while it was closed.

@atk atk requested a review from Vaelatern February 4, 2020 10:24
@Vaelatern Vaelatern closed this in 58f4d59 Feb 6, 2020
@atk
Copy link
Contributor Author

atk commented Feb 6, 2020

I have answered every of your change requests. May I ask why you closed the PR, @Vaelatern?

@Vaelatern
Copy link
Member

Look closer.

@atk
Copy link
Contributor Author

atk commented Feb 6, 2020

I'm really sorry to waste your time, but what am I supposed to look at?

You asked for musl support, which is not possible at the moment but will be added once it is. You asked me to remove the install script, which I did. You asked be to reopen the previous PR, which GitHub won't let me do. What have I missed?

@Duncaen
Copy link
Member

Duncaen commented Feb 6, 2020

It was closed with a commit: 58f4d59.

The PR has been merged manually, but github is not smart enough to notice it if it has been merged/rebased by hand if the hash changed.

@atk
Copy link
Contributor Author

atk commented Feb 6, 2020

Ah, thanks.

@atk atk deleted the feat-volta branch February 6, 2020 18:52
@Vaelatern
Copy link
Member

I merged it, with a few tweaks that did not require a roundtrip, I could just quickly do them myself.

Welcome to the void!

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants