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

Codemirror-Update 5.58.3 #4120

Merged
merged 2 commits into from Nov 28, 2020
Merged

Codemirror-Update 5.58.3 #4120

merged 2 commits into from Nov 28, 2020

Conversation

aeberhard
Copy link
Contributor

Update Vendor von Version 5.51.0 auf 5.58.3
Keine neue Funktionalität hinzugefügt
Lokal und auf Linux-Webspace getestet

Update Vendor von Version 5.51.0 auf 5.58.3
Keine neue Funktionalität hinzugefügt
Lokal und auf Linux-Webspace getestet
@staabm
Copy link
Member

staabm commented Nov 28, 2020

ohh super. diese abhängigkeit hab ich überhaupt nicht aufm schirm. danke dir.

können wir das update von codemirror auch in https://github.com/redaxo/redaxo/blob/master/.tools/bin/update-all via bash mit aufnehmen (separater PR)?

@staabm staabm added the dependencies Pull requests that update a dependency file label Nov 28, 2020
@gharlan gharlan added this to the REDAXO 5.12 milestone Nov 28, 2020
Copy link
Member

@gharlan gharlan 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 grob angetestet.

Ich nehme das Update hier so erstmal rein. Danke für's Aktualisieren!

Automatisieren wäre nicht verkehrt. Wobei dann aber sichergestellt werden müsste, dass sich das Zip wirklich nur bei neuer Version ändert. (Ich weiß nicht, ob zips exakt gleich werden bei gleichem Inhalt? Ggf. spielen Dateizeitstempel da eine Rolle etc.)

@gharlan gharlan added the automerge Automatisch PR rebasen und mergen label Nov 28, 2020
@kodiakhq kodiakhq bot merged commit db6bfb2 into redaxo:master Nov 28, 2020
@aeberhard
Copy link
Contributor Author

Ich weiss nicht ob das mit dem automatisieren hier viel Sinn macht.
Der codemirror ist ziemlich abgestrippt, also hauptsächlich Sprachunterstützungen und m.M. nach überflüssige Module sind wg. der Dateigröße entfernt. Zusätzlich sind die JS+CSS Dateien nicht minified, das mache ich auch immer manuell.
Und dann das ganze in ein Zip-Archiv packen.

@staabm
Copy link
Member

staabm commented Nov 29, 2020

Ist irgendwo grob dokumentiert was da alles aktuell an manuellem doing notwendig ist?

@aeberhard
Copy link
Contributor Author

Nein, ist nur in meinem Schädel :)
Ich kontrolliere ob was brauchbares an neuen Modulen dabei ist.
Alles überflüssige wird gelöscht. Sprachunterstützungen nur die die ich für notwendig halte.
Und CSS + JS manuell minify.
So wird codemirror von 1.458kb auf 358kb eingedampft

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Automatisch PR rebasen und mergen dependencies Pull requests that update a dependency file
Development

Successfully merging this pull request may close these issues.

None yet

3 participants