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

adiciona parametro para inclusao de imagem de capa na geracao do livro e markdown ensinando a gerar o ebook #65

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

fabricio
Copy link

No description provided.

@ramalho
Copy link
Contributor

ramalho commented Oct 18, 2023

Muito grato pela suas contribuições, @fabricio.

Antes de aprovar esste PR, eu gostaria que você fizesse as seguintes mudanças, por gentileza:

  1. Separe as correções não relacionadas ao tema principal (imagem de capa) em outro PR. Por exemplo, você pode fazer outro PR só com as correções dos capítulos 13 e 14. Motivo: cada PR deve ter apenas um tema, idealmente.

  2. Não inclua o epub no PR. Motivo: binários gerados pelo código do repositório não devem ser colocados no repositório. Se o objetivo é facilitar o download, o github tem outro lugar para isso.

@fabricio
Copy link
Author

fabricio commented Oct 18, 2023

farei i isso @ramalho. ainda faltam uns 5 ancoras que eu identifiquei gerando o epub vou commitar eles e logo depois separo os PR

asciidoctor: WARNING: livro.adoc: invalid reference to unknown anchor: metaprog_part
asciidoctor: WARNING: livro.adoc: invalid reference to unknown anchor: classes_protocols_part
asciidoctor: WARNING: livro.adoc: invalid reference to unknown anchor: classes_protocols_part
asciidoctor: WARNING: livro.adoc: invalid reference to unknown anchor: metaprog_part
asciidoctor: WARNING: livro.adoc: invalid reference to unknown anchor: function_objects_part
asciidoctor: WARNING: livro.adoc: invalid reference to unknown anchor: assim
asciidoctor: WARNING: livro.adoc: invalid reference to unknown anchor: classes_protocols_part
asciidoctor: WARNING: livro.adoc: invalid reference to unknown anchor: function_objects_part
asciidoctor: WARNING: livro.adoc: invalid reference to unknown anchor: function_objects_part
asciidoctor: WARNING: livro.adoc: invalid reference to unknown anchor: classes_protocols_part
asciidoctor: WARNING: livro.adoc: invalid reference to unknown anchor: metaprog_part
asciidoctor: WARNING: livro.adoc: invalid reference to unknown anchor: herança
asciidoctor: WARNING: livro.adoc: invalid reference to unknown anchor: herança
asciidoctor: WARNING: livro.adoc: invalid reference to unknown anchor: preface_sec

@ramalho
Copy link
Contributor

ramalho commented Oct 18, 2023

asciidoctor: WARNING: livro.adoc: invalid reference to unknown anchor: assim

A âncora assim não deve ser alterada. Ela serve para deixar visível para os leitores quando pode haver uma âncora quebrada, OK?

@fabricio
Copy link
Author

pronto. deixei somente os commits da capa/ ensiando a gerar o ebook. vou criar um novo pr com as correcoes das ancoras assim que esse for fechado

@fabricio fabricio changed the title adiciona parametro para inclusao de imagem de capa na geracao do livro adiciona parametro para inclusao de imagem de capa na geracao do livro e markdown ensinando a gerar o ebook Oct 18, 2023
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