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

Magerun binaries version bump #414

Merged
merged 1 commit into from
Sep 26, 2019
Merged

Magerun binaries version bump #414

merged 1 commit into from
Sep 26, 2019

Conversation

Igloczek
Copy link
Contributor

I had to manually update magerun2 binaries to make it compatible with M2.3.2 and saw that this issue can be easily fixed without any manual work just by always using the latest version.
As a bonus, you will not have to do a release of Valet to get some updates/fixes from Magerun.

@Neodork
Copy link
Collaborator

Neodork commented Sep 19, 2019

@Igloczek How will we determine the shasum of the latest version? I think this will always fail.

@Igloczek
Copy link
Contributor Author

Damn, you are right, didn't saw that shasum in code 🙈
I'll change this PR to just make an update to the latest versions and will be good enough for now.

@Igloczek Igloczek changed the title Use latest magerun version instead strict numbers Magerun binaries version bump Sep 19, 2019
@Neodork
Copy link
Collaborator

Neodork commented Sep 25, 2019

@Igloczek This looks good. Can you squash your commits and target the 2.x branch? If all tests pass we can merge this.

@Igloczek Igloczek changed the base branch from master to 2.x September 25, 2019 08:01
@Igloczek
Copy link
Contributor Author

@Neodork Done.

@Neodork
Copy link
Collaborator

Neodork commented Sep 26, 2019

binaries] Installing binaries
[binaries] magerun not found, installing from: https://files.magerun.net/n98-magerun-1.103.1.phar
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
 14 5383k   14  783k    0     0  1861k      0  0:00:02 --:--:--  0:00:02 1861k
100 5383k  100 5383k    0     0  6470k      0 --:--:-- --:--:-- --:--:-- 6470k
[binaries] magerun installed to: /usr/local/bin/magerun
[binaries] magerun2 not found, installing from: https://files.magerun.net/n98-magerun2-3.2.0.phar
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

Looks good to me, merging.

@Neodork Neodork merged commit 2705275 into weprovide:2.x Sep 26, 2019
@Igloczek Igloczek deleted the patch-1 branch September 26, 2019 13:35
@Neodork
Copy link
Collaborator

Neodork commented Oct 2, 2019

Changelog entries:

  • Changed magerun to version 1.103.1
  • Changed magerun2 to version 3.2.0

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