Skip to content

Conversation

@edgarrmondragon
Copy link
Contributor

No description provided.

@mergify
Copy link
Contributor

mergify bot commented Jun 24, 2024

⚠️ No release notes detected. Please make sure to use reno to add a changelog entry.

@edgarrmondragon edgarrmondragon marked this pull request as ready for review June 24, 2024 17:11
@jd
Copy link
Owner

jd commented Jun 26, 2024

You might want to add this to the Mergify configuration too.

@edgarrmondragon
Copy link
Contributor Author

You might want to add this to the Mergify configuration too.

Done!

jd
jd previously approved these changes Jun 27, 2024
@mergify mergify bot dismissed jd’s stale review July 3, 2024 22:00

Pull request has been modified.

Copy link
Owner

@jd jd left a comment

Choose a reason for hiding this comment

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

Do we need to update classifiers to in setup.cfg?

@edgarrmondragon
Copy link
Contributor Author

Do we need to update classifiers to in setup.cfg?

Yeah, why not? Done!

jd
jd previously approved these changes Jul 6, 2024
@jakkdl
Copy link
Contributor

jakkdl commented Oct 23, 2024

Would be great if this could be merged for the sake of other PRs :)

Co-authored-by: John Litborn <[email protected]>
@mergify mergify bot dismissed jd’s stale review October 23, 2024 13:21

Pull request has been modified.

jd
jd previously approved these changes Oct 23, 2024
@mergify mergify bot dismissed jd’s stale review February 17, 2025 19:55

Pull request has been modified.

Co-authored-by: Bouchafa Lotfi Rafik <[email protected]>
@edgarrmondragon edgarrmondragon requested a review from jd February 18, 2025 05:22
@jd jd added the no-changelog No changelog needed label Feb 18, 2025
@jd jd merged commit 3203359 into jd:main Feb 18, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog No changelog needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants