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

Added Codefresh.yml pipeline #4235

Merged
merged 2 commits into from
Aug 23, 2018
Merged

Added Codefresh.yml pipeline #4235

merged 2 commits into from
Aug 23, 2018

Conversation

homdx
Copy link
Contributor

@homdx homdx commented Aug 7, 2018

Added Codefresh.yml pipeline

@moneromooo-monero
Copy link
Collaborator

What is it, what does it do ?

@homdx
Copy link
Contributor Author

homdx commented Aug 8, 2018

https://codefresh.io/pricing/

This is service for build docker images.
I tested WORKFLOW with Inline YAML Use YAML from Repository

@moneromooo-monero
Copy link
Collaborator

Can you put it in some directory in contrib please ? This way it doesn't spam the root directory. Moving the docker file would also be nice, not sure why it got added in the root.

@homdx
Copy link
Contributor Author

homdx commented Aug 8, 2018

  1. Moved to folder contrib
  2. if you want the file Docker to be moved to another folder, in this folder added all files from root folder becouse docker make with command:
    COPY . .
    (copy all files from root folder to docker). Docker can't make like this command:
    COPY ../ .

Sea answer on; https://stackoverflow.com/questions/24537340/docker-adding-a-file-from-a-parent-directory

@moneromooo-monero
Copy link
Collaborator

Thanks. The other file can stay if it's causing trouble.

@luigi1111 luigi1111 merged commit 13a43fc into monero-project:master Aug 23, 2018
luigi1111 added a commit that referenced this pull request Aug 23, 2018
3411326 Docker: updated cmake version (homdx)
13a43fc Added Codefresh.yml pipeline (homdx)
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.

3 participants