Skip to content

[Merged by Bors] - fix: curl is not needed for all commands#1956

Closed
arthurpaulino wants to merge 1 commit intomasterfrom
ap/fix-curl-warn-error
Closed

[Merged by Bors] - fix: curl is not needed for all commands#1956
arthurpaulino wants to merge 1 commit intomasterfrom
ap/fix-curl-warn-error

Conversation

@arthurpaulino
Copy link
Copy Markdown
Contributor

@arthurpaulino arthurpaulino commented Jan 30, 2023

curl is not needed for all cache commands, as mentioned here

This PR also changes mk to pack so it's consistent with unpack

@gebner
Copy link
Copy Markdown
Member

gebner commented Jan 30, 2023

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Jan 30, 2023
bors bot pushed a commit that referenced this pull request Jan 30, 2023
`curl` is not needed for all `cache` commands, as mentioned [here](https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/New.20Project.20that.20uses.20MathLib4/near/324783543)

This PR also changes `mk` to `pack` so it's consistent with `unpack`
@bors
Copy link
Copy Markdown

bors bot commented Jan 30, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title fix: curl is not needed for all commands [Merged by Bors] - fix: curl is not needed for all commands Jan 30, 2023
@bors bors bot closed this Jan 30, 2023
@bors bors bot deleted the ap/fix-curl-warn-error branch January 30, 2023 23:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has been sent to bors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants