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

feat: add pdf conversions #4

Merged
merged 3 commits into from
Oct 24, 2023
Merged

feat: add pdf conversions #4

merged 3 commits into from
Oct 24, 2023

Conversation

jgabaut
Copy link
Member

@jgabaut jgabaut commented Oct 23, 2023

Patch as mentioned in here.

   Patch as mentioned in [here](csunibo/build-image#3).
@jgabaut
Copy link
Member Author

jgabaut commented Oct 23, 2023

Sembra che il .pdf aggiunto (con lo stesso nome del file originale) venga sovrascritto dal workflow? Dovrei rinominarlo?

Edit: credo solo quello derivato dal .pptx sia sovrascritto.

@jgabaut
Copy link
Member Author

jgabaut commented Oct 23, 2023

Nel dubbio, ho rimosso il .pptx in un secondo commit.

@jgabaut
Copy link
Member Author

jgabaut commented Oct 23, 2023

A quanto pare anche il .odt sovrascrive il .pdf con lo stesso nome, ho rimosso anche quello.

Mi chiedo se rimuovere il file "originale" perda di potenziale in quanto il .pdf è "meno editabile" ? Non so bene cosa sto dicendo.

@foxyseta
Copy link
Member

Ciao! Sì, proprio come di norma si vuole che git tracci i sorgenti e non i binari, così noi vorremmo che git tracci gli odt/ppt e non i pdf (questa norma è documentata in https://csunibo.github.io/wiki/raccolte-di-risorse/). Tuttavia, questa PR è un'eccezione siccome la compilazione dgli odt/ppt al momento da quel problema con i caratteri. Quindi se mi dai il permesso io mergerei tutto così com'è, e poi puoi ricaricare l'odt e il ppt quando avremo risolto csunibo/build-image#3.

@jgabaut
Copy link
Member Author

jgabaut commented Oct 23, 2023

Capisco. Per me va benissimo, LGTM!

@foxyseta foxyseta merged commit d143c8b into csunibo:main Oct 24, 2023
3 checks passed
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