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

Add ./dev bash script for dev actions #1083

Merged
merged 21 commits into from Sep 10, 2018
Merged

Add ./dev bash script for dev actions #1083

merged 21 commits into from Sep 10, 2018

Conversation

rafalp
Copy link
Owner

@rafalp rafalp commented Sep 1, 2018

This PR adds ./dev bash script that provides single point of entry for running dev commands such as creating dev project, clearing it, making messages, running tests, etc. ect.

Fixes #1079

Some cool stuff:

Running ./dev init will cause following to happen:

  • if port 8000 is already taken, we tell user to shut down whatever is running on it and try again
  • if port 5432 is already taken, we tell user to stop system-running PostgreSQL instance and try again
  • if there are remaining files from old ./dev init present, we kindly ask user to run ./dev clear and try again
  • we install everything needed to run misago, create the devproject with required dirs, expose manage.py, migrate the DB, create super, and run dev server
  • we display short welcome message to user telling them to go

COMMANDS

  • Initialize new dev project
  • Delete existing dev project
  • Rebuild docker
  • Make messages
  • Compile messages
  • Pull from transifex
  • Push to transifex
  • Shortcut for running all tests
  • Shortcut for running tests in module
  • Expose psql
  • Shortcut for docker-compose run --rm misago
  • Shortcut for bash
  • Shortcut for manage.py

EXTRA

  • In some commands check if they are ran in docker before continuing
  • Display some help text in default run
  • Display guide for next step after ./dev init completes
  • Add help for connecting with datebase
  • Add command for getting post-init intro after already initializing the dev project
  • Update readme
  • Remove deprecated bash scripts
  • Remove extras directory

@rafalp rafalp added enhancement Improvement to existing feature or dev experience area: tooling This issue involves Docker or Bash script labels Sep 1, 2018
@rafalp rafalp added this to the 0.19.2 milestone Sep 1, 2018
@rafalp rafalp self-assigned this Sep 1, 2018
@rafalp
Copy link
Owner Author

rafalp commented Sep 2, 2018

CC @einarf could you find a minute and give this work a look? There's still some work to be done here, but features look like they are done.

This is also sort of test run for proper appctl script that I plan to write for misago-docker down the road.

@einarf
Copy link
Contributor

einarf commented Sep 2, 2018

At first glance this looks very reasonable. You have at least the features I would like to see.

@rafalp rafalp merged commit 4802060 into master Sep 10, 2018
@rafalp rafalp deleted the consolidate-scripts branch September 10, 2018 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: tooling This issue involves Docker or Bash script enhancement Improvement to existing feature or dev experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants