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

upgrade to plone 6.0.13 #82

Merged
merged 16 commits into from
Sep 18, 2024
Merged

upgrade to plone 6.0.13 #82

merged 16 commits into from
Sep 18, 2024

Conversation

mamico
Copy link
Contributor

@mamico mamico commented Jul 5, 2024

No description provided.

@cekk
Copy link
Member

cekk commented Jul 15, 2024

+1 for this..it also fix a fuzzy translation in all description fields.

@mamico mamico marked this pull request as ready for review July 15, 2024 14:31
@luca-bellenghi
Copy link
Member

Va benissimo. lascio questo commento come "visto".
Per altre scadenze nel breve non riesco a mergiarlo e provarlo. la preoccupazione deriva dal fatto che poco tempo fa avevo già fatto un tentativo e visto che c'era da aggiornare alcune cose su restapi (la versione) e design.plone.contenttypes (alcune cose non funzionavano più, se ricordo bene per import non più validi).
Appena possibile verrà tenuta in considerazione.

@mamico mamico changed the title upgrade to plone 6.0.11.1 upgrade to plone 6.0.12 Aug 6, 2024
@mamico mamico changed the title upgrade to plone 6.0.12 upgrade to plone 6.0.13 Sep 10, 2024
@mamico
Copy link
Contributor Author

mamico commented Sep 10, 2024

ho aggiunto i test per design.plone.policy e design.plone.contenttypes, in locale policy si rompe su un test

@mamico mamico requested a review from cekk September 10, 2024 07:49
@mamico
Copy link
Contributor Author

mamico commented Sep 12, 2024

Va benissimo. lascio questo commento come "visto". Per altre scadenze nel breve non riesco a mergiarlo e provarlo. la preoccupazione deriva dal fatto che poco tempo fa avevo già fatto un tentativo e visto che c'era da aggiornare alcune cose su restapi (la versione) e design.plone.contenttypes (alcune cose non funzionavano più, se ricordo bene per import non più validi). Appena possibile verrà tenuta in considerazione.

@luca-bellenghi hai qualche informazione in più sui problemi che riscontravi? traceback, ...

@mamico
Copy link
Contributor Author

mamico commented Sep 12, 2024

ho aggiunto i test per design.plone.policy e design.plone.contenttypes, in locale policy si rompe su un test

il test su cui si rompeva è una traduzione. c'è invece il problema che si rompono i test di plone (plone.app.portlets !) quando si aggiungono i test di design.*

@luca-bellenghi
Copy link
Member

Va benissimo. lascio questo commento come "visto". Per altre scadenze nel breve non riesco a mergiarlo e provarlo. la preoccupazione deriva dal fatto che poco tempo fa avevo già fatto un tentativo e visto che c'era da aggiornare alcune cose su restapi (la versione) e design.plone.contenttypes (alcune cose non funzionavano più, se ricordo bene per import non più validi). Appena possibile verrà tenuta in considerazione.

@luca-bellenghi hai qualche informazione in più sui problemi che riscontravi? traceback, ...

Per la versione 6.0.11 stavo guardando anche io e dovrei già aver fissato un paio di cose sia in design.plone.policy che in design.plone.contenttypes. Questo supera i problemi precedenti.

Adesso, se cambio il version nei due pacchetti usando il branch corrente (quindi plone 6.0.13):
dpc: Total: 272 tests, 0 failures, 0 errors, 0 skipped in 49.264 seconds.
dpp: Total: 27 tests, 0 failures, 0 errors, 0 skipped in 18.249 seconds.

I test dei singoli pacchetti sono tutti verdi.
Il buildout di iocomune con Plone 6.0.13 e i main di dct e dpp parte senza problemi e posso aggiornare.

c'è invece il problema che si rompono i test di plone (plone.app.portlets !) quando si aggiungono i test di design.*
Non sono sicuro di aver capito come testi il problema che hai rilevato tu. Ad ogni modo vedo che vai ad escludere dai test-egg plone.app. Secondo me ci sta: questo repo, al di la di tutto, è usato per ambienti che prevedono installazioni volto.

Non so se volevi parlare di altro (nel caso sentiamoci) ma mi sta bene la tua proposta.

@luca-bellenghi luca-bellenghi merged commit d82800e into main Sep 18, 2024
4 checks passed
@luca-bellenghi luca-bellenghi deleted the plone_6011_1 branch September 18, 2024 12:20
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.

3 participants