Skip to content

Run scripts from Makefile#1402

Merged
NicolasMahe merged 19 commits intodevfrom
refactor/makefile
Oct 22, 2019
Merged

Run scripts from Makefile#1402
NicolasMahe merged 19 commits intodevfrom
refactor/makefile

Conversation

@krhubert
Copy link
Copy Markdown
Contributor

@krhubert krhubert commented Oct 9, 2019

  • create Makefile
  • remove mesg-tools and require-mesg-tools scripts
  • move build-engine to Dockerfile
  • remove go-bindata from Dockefile.tools
  • require changelog arg
  • update .gitignore
  • remove Dockerfile.dev
  • move dev to scripts dir

close #1393

TODO: There is no option to pass an argument to dev script right now. I need to figure it out, but it will be done with YAML config update.

I think everything now is in Makefile and it's a lot cleaner

@krhubert krhubert added this to the next milestone Oct 9, 2019
@krhubert krhubert self-assigned this Oct 9, 2019
Comment thread Dockerfile Outdated
@krhubert krhubert force-pushed the refactor/makefile branch 2 times, most recently from b5aab9e to 6061242 Compare October 10, 2019 13:25
- craete Makefile
- remove mesg-tools and require-mesg-tools scripts
- move build-engine to Dockerfile
- remove go-bindata from Dockefile.tools
- require changelog arg
- remove Dockerfile.dev
- move dev to scripts dir
Comment thread Makefile Outdated
Comment thread Dockerfile Outdated
Comment thread Makefile Outdated
Comment thread Makefile Outdated
@krhubert krhubert requested a review from NicolasMahe October 21, 2019 10:58
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 21, 2019

Codecov Report

Merging #1402 into dev will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##              dev    #1402   +/-   ##
=======================================
  Coverage   62.94%   62.94%           
=======================================
  Files          41       41           
  Lines        1773     1773           
=======================================
  Hits         1116     1116           
  Misses        558      558           
  Partials       99       99

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0f0f267...4f2a3f5. Read the comment docs.

@krhubert krhubert requested a review from antho1404 October 21, 2019 10:58
Comment thread .dockerignore
Comment thread Makefile
Comment thread Makefile
Comment thread Makefile
Comment thread Dockerfile Outdated
krhubert and others added 2 commits October 22, 2019 11:08
Move advance version tagging from makefile to circleci
Co-Authored-By: Nicolas Mahé <nicolas@mesg.com>
Comment thread scripts/dev.sh
fi

# create mesg data folder on host machine
mkdir -p $MESG_PATH
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this is still important!

@NicolasMahe NicolasMahe merged commit aa45510 into dev Oct 22, 2019
@NicolasMahe NicolasMahe deleted the refactor/makefile branch October 22, 2019 09:57
@NicolasMahe NicolasMahe added the release:change Pull requests that change something existant label Oct 25, 2019
@NicolasMahe NicolasMahe mentioned this pull request Oct 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactoring release:change Pull requests that change something existant

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve bash dev scripts organisation

3 participants