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 GitHub Action workflow to build the book #50

Merged
merged 4 commits into from
Aug 31, 2020
Merged

Add GitHub Action workflow to build the book #50

merged 4 commits into from
Aug 31, 2020

Conversation

kerolloz
Copy link
Contributor

action run example: https://github.com/kerolloz/api_on_rails/runs/1042903492?check_suite_focus=true
the created release with built versions for all languages and extensions(except MOBI): https://github.com/kerolloz/api_on_rails/releases/tag/untagged-2332728a3393c5c327c1

What do you think @madeindjs ? any changes?

- updated packages & bundler
- removed 'kindlegen'. More at tdtds/kindlegen#42
Copy link
Owner

@madeindjs madeindjs left a comment

Choose a reason for hiding this comment

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

Thank you very much for your work. I was not aware that Github action allow us to do this kind of stuff. I made a small code review with some question.

Once you answer me, I will merge it. I'm so excited to watch result on my repository! 😄 Thanks again.

Gemfile Outdated Show resolved Hide resolved
.github/workflows/ci.yml Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
kerolloz and others added 2 commits August 29, 2020 13:56
Co-authored-by: Alexandre Rousseau <contact@rousseau-alexandre.fr>
@kerolloz kerolloz requested a review from madeindjs August 30, 2020 01:28
…mber

Co-authored-by: Alexandre Rousseau <contact@rousseau-alexandre.fr>
@madeindjs madeindjs merged commit c5bfe70 into madeindjs:master Aug 31, 2020
@madeindjs
Copy link
Owner

@kerolloz I just merged. I made a little fix but it works 🎆 .

Thank you again!

@kerolloz kerolloz deleted the ci-cd branch August 31, 2020 10:58
@kerolloz
Copy link
Contributor Author

Yesterday, I finished reading the book. I learned a lot. Thank you @madeindjs.
I think I will be making more contributions. 😃

@kerolloz
Copy link
Contributor Author

Here is what I have built. Take a look at the README https://github.com/kerolloz/marketplace_api#readme

@madeindjs
Copy link
Owner

Oh Great. Feel free to make PR on marketplace API too 😉

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.

2 participants