Skip to content

fix(): Streamline setup process of local dev environment - #88

Merged
rodmgwgu merged 2 commits into
quince.masterfrom
fix/setupscript
Nov 22, 2024
Merged

rodmgwgu merged 2 commits into
quince.masterfrom
fix/setupscript

Conversation

@rodmgwgu

Copy link
Copy Markdown
Collaborator

Overview

Remove usage of mariadb and building of the OpenEDX image during setup of the local dev enviromnent, as this is no longer needed and just takes longer to set things up.

Context: the usage of mariadb instead of MySQL for the dev environment was made originally because on older versions of OpenEDX, MySQL 5.7 was required, and there was no docker image for that version of MySQL for ARM (which is the platform we use to develop on Apple Silicon Macs). Newer versions of OpenEDX use MySQL 8.0, which does offer docker images for ARM.

We also used to build the image to make it ARM native, but nowadays we can just pull the official image and run it via rosetta which is performant enough for this use case.

Checklist

⚠️ Please make sure to fill this checklist before asking for reviews.

  • Code is correctly formatted and linted
  • Unit tests are updated and are passing
  • PR title follows conventional commits:
    i.e. feat(CUR-###): Feature title
  • PR Title aligns to Semantic-Release supported prefixes (NOTE: prefixes are case sensitive, keep lower-case):
 feat() - Feature (0.X.0)
 fix() - Patch (0.0.X)
 docs() - Patch (0.0.X), will only scope to README changes
 refactor() - Patch (0.0.X)
 revert() - Patch (0.0.X)
 style() - Patch (0.0.X)
  • Check for unused files
  • Check work before asking for reviewers
  • Fix any linting errors
  • SECURITY: No secrets where commited to the repo
  • COMPLIANCE: Commited code is not propietary and adheres to Open Source licensing

@j0mc j0mc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@rodmgwgu
rodmgwgu merged commit 8dba95e into quince.master Nov 22, 2024
@rodmgwgu
rodmgwgu deleted the fix/setupscript branch November 22, 2024 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants