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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃懛 Simple gitaction for auto PDF creation #33

Closed
wants to merge 5 commits into from

Conversation

fanooel
Copy link

@fanooel fanooel commented May 14, 2023

Simple pdf auto creation from MD files (as artifact in ACTION)

Next step:

  1. Implement release strategy- pdf will be created only for releases.
  2. Switch to 'pandoc' or other action for MD to PDF conversion.

@MateuszNaKodach
Copy link

Bardzo dobry pomys艂. W艂a艣nie to pisa艂em :)

@fanooel
Copy link
Author

fanooel commented May 14, 2023

Jak si臋 pojawi koncepcja z releasami i wersjonowaniem (sugeruj臋 auto release z auto wersjami) to mog臋 to przenie艣膰 to pandoca, gdzie informacje o wersji b臋dzie np. w stopce i stronie tytu艂owej.

Je艣li b臋dzie te偶 strategia z branchem devel- mo偶na devele jako wersje robocze te偶 generowa膰.

No i jeszcze mo偶na doda膰 akcj臋 na dodawanie linka do aktualnej wersji PDF w README :) i CHANGELOG

@fanooel
Copy link
Author

fanooel commented May 14, 2023

Na deser- powiadomienia o releasach dla prawnik贸w kochaj膮cych czytanie PDF

@maciejmrozinski
Copy link
Contributor

Widz臋, 偶e jest nas wi臋cej z tym pomys艂em :)

Copy link
Contributor

@maciejmrozinski maciejmrozinski left a comment

Choose a reason for hiding this comment

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

zmiana nazw output dla utrzymania konwencji startowej

.github/workflows/convert-to-pdf.yml Outdated Show resolved Hide resolved
.github/workflows/convert-to-pdf.yml Outdated Show resolved Hide resolved
Co-authored-by: Maciej Mrozi艅ski <maciej.k.mrozinski@gmail.com>
maciejmrozinski

This comment was marked as resolved.

@fanooel
Copy link
Author

fanooel commented May 14, 2023

Ma razie to jest tylko do artefakt贸w, w Actions, wi臋c nie ma co si臋 za bardzo rozdrabnia膰 w konwencji. Ostatecznie powinno to i tak l膮dowa膰 jako artefakt release.

with:
input_dir: wersje tekstowe
output_dir: wersje pdf
images_dir: wersje tekstowe/images

Choose a reason for hiding this comment

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

Maybe stick to Polish and change images to obrazy? It would be nice to create an empty folder and place .gitkeep inside until the 1st image is created

Choose a reason for hiding this comment

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

I would also consider removing empty spaces from file and directory names.
It may cauze unexpected results while used with software not supporting it.

Copy link

@kkuriata kkuriata left a comment

Choose a reason for hiding this comment

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

It would be nice to remove already existing PDFs, because this workflow will create new ones inside wersje pdf folder

.github/workflows/convert-to-pdf.yml Outdated Show resolved Hide resolved
Comment on lines +9 to +10
- 'wersje tekstowe/**.md'
- 'wersje tekstowe/images/**'
Copy link

Choose a reason for hiding this comment

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

Maybe let's stick to polish or english lang.

@czlowiek488
Copy link

czlowiek488 commented May 15, 2023

Mam wra偶enie 偶e warto by by艂o rozwa偶y膰 zmian臋 konwencji po艂o偶enia plik贸w.
O ile nie spotka艂em si臋 wcze艣niej z projektami ustaw na githubie, to mam wra偶enie 偶e powszechnie znan膮 konwencj膮 jest trzymanie plik贸w 藕r贸d艂owych w roocie projektu lub folderze "src", a wygenerowane pliki w osobnym katalogu.

@MateuszNaKodach
Copy link

Mam wra偶enie 偶e warto by by艂o rozwa偶y膰 zmian臋 konwencji po艂o偶enia plik贸w. O ile nie spotka艂em si臋 wcze艣niej z projektami ustaw na githubie, to mam wra偶enie 偶e powszechnie znan膮 konwencj膮 jest trzymanie plik贸w 藕r贸d艂owych w roocie projektu lub folderze "src", a wygenerowane pliki w osobnym katalogu.

Jednak trzeba pami臋ta膰, 偶e nie tylko programi艣ci s膮 adresatami tego repo :) Jednak偶e 艂atwiej by pewnie by艂o gdyby pliki .MD by艂y jako艣 艂atwiej dost臋pne czy opisane, bo to w nich powinno si臋 proponowa膰 zmiany. PDFy w艂a艣nie w og贸le mo偶na pewnie wywali膰 i publikowa膰 z tego doc (je艣li osoby mniej techniczne dadz膮 rad臋).

@czlowiek488
Copy link

czlowiek488 commented May 15, 2023

@MateuszNaKodach Po twoim komentarzu zastanawiam si臋 czy nie lepsz膮 opcj膮 by艂oby wrzucanie plik贸w na jaki艣 hosting pdf'贸w. W ten spos贸b osoby kt贸re chc膮 przeczyta膰 mia艂yby jeszcze pro艣ciej.
Nie sprawdza艂em takiego rozwi膮zania wcze艣niej, ale - https://fliphtml5.com/ wygl膮da sensownie.
Ewentualnie customowa aplikacja, ale to ju偶 by艂yby koszta.

@pm4rcin
Copy link

pm4rcin commented May 15, 2023

A mo偶e po prostu stworzy膰 stronk臋 z pomoc膮 Github pages (ewentualnie dokupi膰 domen臋, 偶eby w wyszukiwarce wygl膮da艂o profesjonalnie) i tam by si臋 wrzuca艂o automatycznie pdfy z github actions? Oczywi艣cie to by wymaga艂o, 偶eby front-endowcy wymy艣lili dobry layout. 呕e np. jedna podstrona na robocze wersje (devel), co艣 na te otagowane na Githubie. No i przede wszystkim 偶eby to by艂a statyczna strona bez Javascriptu i fajerwerk贸w (jaki艣 static site generator, albo czysty html i css (tailwind?)). Mo偶e w sumie lepiej stworzy膰 issue, 偶eby tu nie 艣mieci膰? Co o tym my艣licie?

Co-authored-by: Bartosz Pazdur <b.pazdur@yahoo.com>
@piotrminkina
Copy link

Co do koncepcji z wydaniami i wersjonowaniem:

- uses: actions/checkout@v3
- uses: baileyjm02/markdown-to-pdf@v1.2.0
with:
input_dir: wersje tekstowe
Copy link
Collaborator

Choose a reason for hiding this comment

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

previous versions of this action accepted the input_dir input. This is still accepted as input for backwards compatibility, but passing a directory as input_path now carries out the same functionality.

https://github.com/BaileyJM02/markdown-to-pdf#github-action-inputs

output_dir: wersje pdf
images_dir: wersje tekstowe/images
# for example <img src="./images/file-name.png">
image_import: ./images
Copy link
Collaborator

Choose a reason for hiding this comment

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

bior膮c pod uwag臋, 偶e w projekcie nie ma 偶adnych obrazk贸w -> mo偶e konfiguracj臋聽dla obrazk贸w mo偶na pomin膮膰?

@dotevo
Copy link

dotevo commented May 16, 2023

PDF to format binarny i nie jest zbyt wygodny do wersjonowania poniewa偶 nie da si臋 zobaczy膰 na nim git diff pomi臋dzy wersjami. Czy jest sens pcha膰 to do Git repo? Mo偶e lepiej trzyma膰 to w release z wygenerowan膮 sum膮 kontroln膮 i tyle - to troch臋 tak jakby pcha膰 wygenerowane pliki "exe" na githuba.

@Jakubk15
Copy link

A mo偶e po prostu stworzy膰 stronk臋 z pomoc膮 Github pages (ewentualnie dokupi膰 domen臋, 偶eby w wyszukiwarce wygl膮da艂o profesjonalnie) i tam by si臋 wrzuca艂o automatycznie pdfy z github actions? Oczywi艣cie to by wymaga艂o, 偶eby front-endowcy wymy艣lili dobry layout. 呕e np. jedna podstrona na robocze wersje (devel), co艣 na te otagowane na Githubie. No i przede wszystkim 偶eby to by艂a statyczna strona bez Javascriptu i fajerwerk贸w (jaki艣 static site generator, albo czysty html i css (tailwind?)). Mo偶e w sumie lepiej stworzy膰 issue, 偶eby tu nie 艣mieci膰? Co o tym my艣licie?

VitePress raczej by si臋 sprawdzi艂

@MacDada
Copy link
Collaborator

MacDada commented May 16, 2023

Wie kto艣 mo偶e jak ustawi膰 czcionk臋 szeryfow膮 dla generowanego PDFa? @fanooel ?

@gajosadrian
Copy link

Osobi艣cie uwa偶am, 偶e czcionki szeryfowe s膮 jednymi z gorszymi do czytania dlatego pozosta艂bym przy tym co jest. A finalnie to i tak te Ustawy b臋d膮聽r臋cznie obrabiane w Word 馃檪.

Aktualnie u偶ywany skrypt baileyjm02/markdown-to-pdf@v1.2.0 nie umo偶liwia bezpo艣redniego ustawienia czcionki, ale mo偶emy go zast膮pi膰 innym narz臋dziem, kt贸re obs艂uguje niestandardowe czcionki.

Nie jestem ekspertem w GitHub Workflows ale mo偶na spr贸bowa膰 z tym:

- name: Install wkhtmltopdf
  run: sudo apt-get install -y wkhtmltopdf

- name: Install fonts
  run: sudo apt-get install -y fonts-dejavu-core

- name: Convert to PDF
  run: |
    cd wersze\ tekstowe
    find . -name '*.md' -type f -exec sh -c 'wkhtmltopdf --enable-local-file-access --margin-top 0mm --margin-right 0mm --margin-bottom 0mm --margin-left 0mm --page-size A4 --no-background --no-outline --dpi 300 --footer-spacing 0 -T 0mm -R 0mm -B 0mm -L 0mm --header-spacing 0 -t pdf --encoding utf-8 --quiet --quiet "{}" "../wersze pdf/{}.pdf"' \;

@fanooel
Copy link
Author

fanooel commented May 17, 2023

@gajosadrian jak ju偶 to lepiej u偶y膰 pandoca, gdzie mo偶esz zrobi膰 w艂asny template w TEXie i nawet pododawa膰 za艂膮czniki, spisy tre艣ci, nag艂贸wki, stopki i co tam chcesz. Od biedy mo偶na tym te偶 eksportowa膰 do odt czy worda :)

@Jakubk15
Copy link

Ewentualnie mo偶na generowa膰 PDF'y za pomoc膮 tego skryptu w GitHub Actions, ale nie wersjonowa膰 tego w repozytorium, tylko jako artefakt w Actionsach i przy wydaniu wersji (release).

MacDada added a commit to MacDada/Ustawy-o-PIT that referenced this pull request May 21, 2023
MacDada added a commit to MacDada/Ustawy-o-PIT that referenced this pull request Jun 12, 2023
@MacDada
Copy link
Collaborator

MacDada commented Jun 14, 2023

Wdro偶yli艣my w ko艅cu nieco bardziej rozbudowan膮 wersj臋 z automatycznymi release i generowaniem PDF贸w przy pomocy Pandoca i LaTeXa (daj膮 wi臋ksze mo偶liwo艣ci konfiguracji).

Dzi臋kujemy za propozycje i sugestie!

@MacDada MacDada closed this Jun 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
techniczne Technikalia, a nie merytoryka
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet