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

QOL Updates - Größtenteils Aufräumen plus kleine bugfixes #126

Merged
merged 10 commits into from
Mar 4, 2022

Conversation

photovoltex
Copy link
Collaborator

Overall Aufräumarbeiten damit alles etwas besser lesbar ist (hoffentlich).
Bitte euch ordentlich drüber zu schauen, da ich zwar schon aufgepasst haben, aber die letzten Änderungen vorallem nicht so stark mitbekommen habe.

Btw. würde gerne hiermit #78 schließen insofern niemand etwas dagegen hat? Theoretisch ist es keine wirkliche Verbesserung seitens der Geschwindigkeit. Man kann noch einen fastbuild bereitstellen, aber ehrlich gesagt... muss das Template so schnell bauen, oder?

Feedback wie immer gerne gesehen :D

@photovoltex
Copy link
Collaborator Author

photovoltex commented Feb 11, 2022

Files die weniger wichtige Änderungen haben. 9/15

.vscode/settings.json              => new words added
.editorconfig                      => added one setting
.vscode/tex_snippets.code-snippets => added one snippet
abk.tex                            => indentation changes
build/tests/main.tex               => change because cleanup of literatur.bib
build/components/com.tex           => it's gone :D
build/components/sperrvermerk.tex  => indentation change
config.tex                         => indentation changes + single rename
literatur.bib                      => cleanup of literatur.bib

@RvNovae RvNovae self-assigned this Feb 11, 2022
@RvNovae RvNovae added the enhancement New feature or request label Feb 11, 2022
Copy link
Collaborator

@ZeroPointMax ZeroPointMax left a comment

Choose a reason for hiding this comment

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

Ich habe es für ein Approval noch nicht getestet, aber sieht schonmal gut aus ^^

.gitignore Outdated Show resolved Hide resolved
build/components/package.config.tex Show resolved Hide resolved
build/template.tex Outdated Show resolved Hide resolved
build/components/deckblatt.tex Outdated Show resolved Hide resolved
build/components/setup.tex Show resolved Hide resolved
@ZeroPointMax
Copy link
Collaborator

Btw. würde gerne hiermit #78 schließen insofern niemand etwas dagegen hat? Theoretisch ist es keine wirkliche Verbesserung seitens der Geschwindigkeit. Man kann noch einen fastbuild bereitstellen, aber ehrlich gesagt... muss das Template so schnell bauen, oder?

Ja, ohne konkret das Bottleneck zu kennen, ist die Build Time schwer zu verbessern.

Bzgl. muss das so schnell: Ja, mein Laptop wird getoastet, bis der Intel Turbo Boost ausläuft :D

@photovoltex
Copy link
Collaborator Author

Btw. würde gerne hiermit #78 schließen insofern niemand etwas dagegen hat? Theoretisch ist es keine wirkliche Verbesserung seitens der Geschwindigkeit. Man kann noch einen fastbuild bereitstellen, aber ehrlich gesagt... muss das Template so schnell bauen, oder?

Ja, ohne konkret das Bottleneck zu kennen, ist die Build Time schwer zu verbessern.

Bzgl. muss das so schnell: Ja, mein Laptop wird getoastet, bis der Intel Turbo Boost ausläuft :D

Hmm... ich glaube das liegt am Ende einfach an den ganzen Setups und Automatisierungen...
Wird also wahrscheinlich nur mit einer Art fastbuild möglich sein (importiert dann nur noch nötiges und Setups werden auch auf das minimum reduziert)

ZeroPointMax and others added 3 commits February 14, 2022 22:39
consolidate indentation (qol PR)

scheinbar vergessen zu mergen?
Copy link
Owner

@RvNovae RvNovae left a comment

Choose a reason for hiding this comment

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

Erstmal vielen Dank für die Mühen!
Einige Stellen schauen mittlerweile wirklich deutlich besser aus :D

Ein paar Anmerkungen habe ich noch, scheint aber soweit erstmal alles zu gehen.

Das Problem mit der Indentation bei der Freigabe sollte nicht am PR hier liegen, das muss mal separat gemacht werden.

.vscode/settings.json Show resolved Hide resolved
.vscode/settings.json Show resolved Hide resolved
.vscode/settings.json Show resolved Hide resolved
.editorconfig Show resolved Hide resolved
build/components/package.config.tex Outdated Show resolved Hide resolved
build/components/package.config.tex Outdated Show resolved Hide resolved
build/components/setup.tex Outdated Show resolved Hide resolved
.gitignore Outdated Show resolved Hide resolved
build/components/setup.tex Show resolved Hide resolved
build/components/setup.tex Outdated Show resolved Hide resolved
Co-authored-by: Max <[email protected]>
Co-authored-by: Oliver Kogel <[email protected]>
.gitignore Outdated Show resolved Hide resolved
@photovoltex photovoltex requested a review from RvNovae March 2, 2022 08:57
.gitignore Show resolved Hide resolved
Copy link
Owner

@RvNovae RvNovae left a comment

Choose a reason for hiding this comment

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

So genug reviewed. Das passt.

Vielen Dank!

@RvNovae RvNovae merged commit 8b1c48f into main Mar 4, 2022
@RvNovae RvNovae deleted the qol-another branch March 4, 2022 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants