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

Update available version of Mariadb to 11.3 #53

Merged

Conversation

uberhacker
Copy link
Collaborator

Also ran npm audit fix to remove vulnerability warnings.

5 vulnerabilities (1 moderate, 3 high, 1 critical)

Example output:

lando ssh -s defaults -c "mysql -V | grep 11.3.2"
mysql from 11.3.2-MariaDB, client 15.2 for Linux (x86_64) using readline 5.1

Copy link

netlify bot commented Apr 6, 2024

👷 Deploy request for lando-mariadb pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 2d9905c

reynoldsalec
reynoldsalec previously approved these changes Apr 8, 2024
@uberhacker
Copy link
Collaborator Author

Thank you for the quick review @reynoldsalec! Should we check the boxes at #51?

@reynoldsalec
Copy link
Sponsor Member

@uberhacker this PR doesn't address #51; that issue is dedicated to updating the version of the @lando/mariadb used across all the recipes that consume it.

@uberhacker
Copy link
Collaborator Author

Oh, so I need to make some more changes to address #51?

@reynoldsalec
Copy link
Sponsor Member

Yeah, you'd need to run those steps against the list of plugins on that issue to update their version of MariaDB. I can probably automate some of that if you want me to generate a bunch of PRs that you can go and review/edit, LMK!

  • Acquia
  • Backdrop
  • Drupal
  • Joomla
  • Laravel
  • Pantheon
  • WordPress

@uberhacker
Copy link
Collaborator Author

Sure, I'm all for automation especially if it relates to version updates!

@uberhacker uberhacker marked this pull request as draft April 8, 2024 21:49
@uberhacker
Copy link
Collaborator Author

In the meantime, I converted this to draft until we can address all the issues in #51.

package.json Outdated Show resolved Hide resolved
reynoldsalec
reynoldsalec previously approved these changes Apr 11, 2024
@uberhacker uberhacker marked this pull request as ready for review April 11, 2024 21:25
@reynoldsalec reynoldsalec merged commit c41dee6 into lando:main Apr 11, 2024
21 checks passed
@reynoldsalec reynoldsalec mentioned this pull request Aug 6, 2024
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