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

repair incomplete commit from last PR (sry) #125

Merged
merged 2 commits into from
Aug 29, 2019
Merged

Conversation

dhavlik
Copy link

@dhavlik dhavlik commented Aug 22, 2019

note to self: when using "commit suggestion" feature of github, keep in mind that there might be more than one such button.
note to self 2: thoroughly test stuff before filing a PR

@mister-roboto
Copy link

@dhavlik thanks for creating this Pull Request and help improve Plone!

To ensure that these changes do not break other parts of Plone, the Plone test suite matrix needs to pass.

Whenever you feel that the pull request is ready to be tested, either start all jenkins jobs pull requests by yourself, or simply add a comment in this pull request stating:

@jenkins-plone-org please run jobs

With this simple comment all the jobs will be started automatically.

Happy hacking!

@icemac
Copy link

icemac commented Aug 22, 2019

@jenkins-plone-org please run jobs

Copy link

@icemac icemac left a comment

Choose a reason for hiding this comment

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

The last two commits should be removed from the PR as they do not belong here.

@dhavlik
Copy link
Author

dhavlik commented Aug 23, 2019

@jenkins-plone-org please run jobs

@icemac
Copy link

icemac commented Aug 26, 2019

@dhavlik Maybe changing the change log entry of the last PR can trick the change log verifier.

@dhavlik
Copy link
Author

dhavlik commented Aug 28, 2019

@jenkins-plone-org please run jobs

@icemac icemac merged commit 8275f5b into plone:master Aug 29, 2019
@mauritsvanrees
Copy link
Member

I have released 6.4.0 with this.
Thanks!

@icemac
Copy link

icemac commented Aug 30, 2019

@mauritsvanrees Thank you!

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