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

WIP: Continuous Deployment! #206

Open
wants to merge 58 commits into
base: master
Choose a base branch
from

Conversation

peternewman
Copy link
Member

@peternewman peternewman commented Jun 20, 2018

Finally! Some progress on #123 .

This currently deploys to https://peter-ola-rdm-app.appspot.com/ every time a build is run on my Travis.

Lots still to-do:

Uncaught TypeError: this.l.ya is not a function
    at X.H (app.js:66)
    at Lc (app.js:43)
    at Y.update (app.js:68)
    at Object.displayCommand (app.js:68)
    at display:190
  • Add a task for it, so it only runs if all other tests pass
  • Probably add index.yaml to repo, and ensure we get all the index entries we need in it. Use @nomis52 one? (See discussion Initial css fixes #10 (diff) )
  • Deploy index.yaml so changes get pushed
  • Update our deploy once Fix GAE deployable files to accept more than one travis-ci/dpl#849 gets merged
  • Switch to dpl v2 stable once it gets release
  • Re-encrypt so it works on the OLA repo?
  • Switch to uploading to rdm.openlighting.org?
  • Version numbers?
  • Work out a solution to "ERROR: (gcloud.app.deploy) INVALID_ARGUMENT: Your app may not have more than 15 versions. Please delete one of the existing versions before trying to create a new version."
  • Secure!

@peternewman peternewman self-assigned this Jun 20, 2018
@peternewman
Copy link
Member Author

Securing maybe done now...

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