Skip to content

Conversation

@Theelx
Copy link
Contributor

@Theelx Theelx commented Dec 9, 2025

Thank you for contributing to jsonpickle! This is just a quick template for you to fill out to make sure that jsonpickle can stay insulated from any legal uncertainty regarding copyrightability of code made by generative AI, and to make sure that we keep the code base easily maintainable.

In order for us to merge your PR, please confirm that all of these boxes are true when applied to this PR and check them if so:

  • If this PR changes any non-documentation portion of the codebase, I have updated the CHANGES.rst file for this change.
  • If this PR fixes a bug or adds a feature, I have added appropriate tests.
  • I have run the tests with pytest and run formatting with black.
  • If generative AI of any kind was used in creating this PR, I have followed the these guideliens:
    • I ensured that this PR is not entirely written by any generative AI model.
    • I ensured that any portions of the code written by any generative AI model are free from copyrighted code.
    • I ensured that I included a Co-authored-by or Assisted-by tag in the footer containing the name of all generative AI models used.
    • I ensured that all code in this PR submitted verbatim from generative AI output is limited to creating/fixing tests/documentation and/or fixing bugs in extensions, and does not impact the core code.

Add a basic PR template for jsonpickle. The PR template contains a general reminder section for things I forget often and an AI section. Ideally, the AI portion should help us limit AI usage to places where it's appropriate (non-mission-critical code), limit the blast radius if it has to be removed for some reason, and so on without completely prohibiting any potential productivity gains. It can be revised in the future to expand allowed usage as AI models get better or shrink allowed usage if necessary, but I think it's good enough for now.

@Theelx Theelx requested a review from davvid December 9, 2025 04:54
Copy link
Member

@davvid davvid left a comment

Choose a reason for hiding this comment

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

Very nicely done, thanks!

@Theelx Theelx merged commit 5e7923c into jsonpickle:main Dec 9, 2025
5 checks passed
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