Skip to content

FIX #122: create build/appstore/ migration dirs in Makefile build target#123

Open
kushvinth wants to merge 1 commit intocytoscape:masterfrom
kushvinth:fix/missing-build-directory
Open

FIX #122: create build/appstore/ migration dirs in Makefile build target#123
kushvinth wants to merge 1 commit intocytoscape:masterfrom
kushvinth:fix/missing-build-directory

Conversation

@kushvinth
Copy link
Copy Markdown

This pull request updates the build process in the Makefile to better support the app store structure by creating necessary directories and __init__.py files. This helps ensure Python recognizes these directories as packages and prevents potential import errors.

Build process improvements:

  • Modified the build target in the Makefile to create new directories for appstore/apps, appstore/download, and appstore/submit_app under the build directory, and to add corresponding __init__.py files to each directory and subdirectory to establish them as Python packages.

Related Issues

The MIGRATION_MODULES setting in settings/local.py points migrations to
build/appstore/{apps,download,submit_app}. The Makefile build target was
missing the creation of these directories and their __init__.py files,
causing makemigrations and migrate to fail with:

  ModuleNotFoundError: No module named 'build.appstore'
  ValueError: Could not locate an appropriate location to create migrations package

This adds mkdir and touch commands to create the required package structure
before running any Django management commands.
@kushvinth
Copy link
Copy Markdown
Author

@coleslaw481, @matthiaskoenig, @sarthak-srivastava, @jingjingbic ,@AlexanderPico and other maintainers PTAL

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.

1 participant