Skip to content

Conversation

YannickJadoul
Copy link
Collaborator

Cleaning up notifications, I came across this comment by @darkdragon-001: #545 (comment)

Bit silly in size as PR, but otherwise, we'll probably forget about it again. If necessary, I can still extend this PR by messing up some markdown and adding "an awesome project" in some places?

@YannickJadoul YannickJadoul force-pushed the valarray-conversion-docs branch from 9f44491 to 38183bc Compare October 4, 2020 00:14
@henryiii
Copy link
Collaborator

henryiii commented Oct 4, 2020

I'm going to merge, partially to see if the changelog needed label gets added. :) I can probably set it to only care about source changes and ignore doc changes, but I want to see it work at least once first! :)

@henryiii henryiii merged commit b70894d into pybind:master Oct 4, 2020
@github-actions github-actions bot added the needs changelog Possibly needs a changelog entry label Oct 4, 2020
@henryiii
Copy link
Collaborator

henryiii commented Oct 4, 2020

🎉

@henryiii henryiii removed the needs changelog Possibly needs a changelog entry label Oct 4, 2020
@henryiii
Copy link
Collaborator

henryiii commented Oct 4, 2020

This line might help:

  - any: ['include/pybind11/**/*.h', 'tools/*.cmake*', '**/CMakeLists.txt', 'pybind11/**/*.py']

@YannickJadoul YannickJadoul deleted the valarray-conversion-docs branch October 4, 2020 14:18
@YannickJadoul
Copy link
Collaborator Author

This line might help:

  - any: ['include/pybind11/**/*.h', 'tools/*.cmake*', '**/CMakeLists.txt', 'pybind11/**/*.py']

Well, better be too careful, I'd say? Looking at a PR and removing "needs changelog" is rather painless?

Anyway, great job, again! :-)

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.

2 participants