Skip to content

Configuring with plone.meta#171

Merged
mauritsvanrees merged 22 commits intomasterfrom
config-with-default-template-999269c3
Mar 19, 2025
Merged

Configuring with plone.meta#171
mauritsvanrees merged 22 commits intomasterfrom
config-with-default-template-999269c3

Conversation

@erral
Copy link
Member

@erral erral commented Mar 17, 2025

I have configured this package with plone/meta because when running the tests of #170 some tests failed with the following error
pkg_resources.DistributionNotFound: The 'plone.namedfile' distribution was not found and is required by plone.app.users, plone.outputfilters, plone.app.vocabularies

and I thought that this could be because of the setuptools thing :/

But now I don't feel confident enough to touch package dependencies (last time I did something similar in a package of the collective we had a lengthy discussion with dependency declaration), so now I am a bit lost :(

I managed to add the dependencies!

@mister-roboto
Copy link

@erral thanks for creating this Pull Request and helping to improve Plone!

TL;DR: Finish pushing changes, pass all other checks, then paste a comment:

@jenkins-plone-org please run jobs

To ensure that these changes do not break other parts of Plone, the Plone test suite matrix needs to pass, but it takes 30-60 min. Other CI checks are usually much faster and the Plone Jenkins resources are limited, so when done pushing changes and all other checks pass either start all Jenkins PR jobs yourself, or simply add the comment above in this PR to start all the jobs automatically.

Happy hacking!

@erral
Copy link
Member Author

erral commented Mar 17, 2025

@jenkins-plone-org please run jobs

@erral
Copy link
Member Author

erral commented Mar 18, 2025

@jenkins-plone-org please run jobs

@erral
Copy link
Member Author

erral commented Mar 18, 2025

@jenkins-plone-org please run jobs

@erral
Copy link
Member Author

erral commented Mar 18, 2025

@jenkins-plone-org please run jobs

Copy link
Member

@mauritsvanrees mauritsvanrees left a comment

Choose a reason for hiding this comment

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

Thanks for tackling this!

@mauritsvanrees mauritsvanrees merged commit 38c2a72 into master Mar 19, 2025
19 checks passed
@mauritsvanrees mauritsvanrees deleted the config-with-default-template-999269c3 branch March 19, 2025 08:42
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.

4 participants