Skip to content
This repository has been archived by the owner on Mar 24, 2023. It is now read-only.

Move examples #202

Merged
merged 1 commit into from
Jul 30, 2018
Merged

Move examples #202

merged 1 commit into from
Jul 30, 2018

Conversation

marccampbell
Copy link
Member

What I Did

Moved fixtures out of examples. Examples will be used for examples on how to use ship, connecting into CI, gitops, etc.

How I Did it

mv && git commit

How to verify it

Check out the examples directory

Description for the Changelog

Picture of a Boat (not required but encouraged)

                   ,:',:`,:'
                __||_||_||_||___
           ____[""""""""""""""""]___
           \ " '''''''''''''''''''' \
    ~~jgs~^~^~^^~^~^~^~^~^~^~^~~^~^~^~^~~^~^

@marccampbell marccampbell requested a review from dexhorthy July 28, 2018 20:24
if serveUIFromAPIDaemon(d) {
g.Use(static.Serve("/", d.WebUIFactory("dist")))
}
g.Use(static.Serve("/", d.WebUIFactory("dist")))
Copy link
Member

Choose a reason for hiding this comment

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

This is gonna break console-ship

Copy link
Member Author

Choose a reason for hiding this comment

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

This will only push to unstable, should we wait to merge until after console-ship has been updated to use docker run instead of docker-compose? Open to safest path here.

Copy link
Member

Choose a reason for hiding this comment

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

Makes sense. Ok to merge then, but do you wanna also change port to 8800?

Copy link
Member Author

Choose a reason for hiding this comment

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

👍

@marccampbell marccampbell merged commit 23d7c6f into replicatedhq:master Jul 30, 2018
@marccampbell marccampbell deleted the move-examples branch July 30, 2018 00:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants