censor/Censor
1
37
Fork
You've already forked Censor
3

feat: add actions to export and import redactions to/from JSON #214

Open
arthaud wants to merge 5 commits from arthaud/Censor:main into main
pull from: arthaud/Censor:main
merge into: censor:main
censor:main
First-time contributor
Copy link
  • Redactions are saved as a list in JSON object
  • JSON object saves a version number for breaking changes
  • Closes #8

DeepSeek-V4-Flash was used to write some of the code, but none of the comments or commit messages

- Redactions are saved as a list in JSON object - JSON object saves a version number for breaking changes - Closes #8 DeepSeek-V4-Flash was used to write some of the code, but none of the comments or commit messages
Redactions are saved as a list in JSON object
JSON object saves a version number for breaking changes
DeepSeek-V4-Flash was used to write some of the code, but none of the
comments or commit messages
fix: filtered imported redactions for duplicates
Some checks failed
/ build-start (pull_request_target) Failing after 11s
/ release-prepare (pull_request_target) Has been skipped
/ checks (pull_request) Has been cancelled
/ merge-conditions (pull_request) Has been cancelled
affc84a9c5

Dear @arthaud, thanks a lot for your contributions and inputs to the project! I‘d like to ask you for some clarifications.

  1. What is the problem you want to solve with this feature? Could you please describe your usage and the limits you encountered?

  2. Thanks for being honest in noting that LLMs were used in writing code. I am afraid that I cannot accept these contributions, because how can you demonstrate that the 1. contributed code can be submitted under the GPL license and 2. it doesn’t violate copyright of LLM training material? See, e.g.,

    Also, the AI agreement of Forgejo (which is the base of the one from Censor) was recently amended to clarify this issue: forgejo/governance@57bf0779be (and I likely should update Censor’s AI agreement, too).

I personally use LLMs as "better search machine" in development sometimes, too, ensuring that all code I write is copyrightable by myself.

Dear @arthaud, thanks a lot for your contributions and inputs to the project! I‘d like to ask you for some clarifications. 1. What is the problem you want to solve with this feature? Could you please describe your usage and the limits you encountered? 2. Thanks for being honest in noting that LLMs were used in writing code. I am afraid that I cannot accept these contributions, because how can you demonstrate that the 1. contributed code can be submitted under the GPL license and 2. it doesn’t violate copyright of LLM training material? See, e.g., - https://www.techtarget.com/searchEnterpriseAI/tip/Examining-the-future-of-AI-and-open-source-software - https://www.recordinglaw.com/us-laws/ai-copyright-laws/ai-generated-code-copyright-infringement/ Also, the AI agreement of Forgejo (which is the base of the one from Censor) was recently amended to clarify this issue: https://codeberg.org/forgejo/governance/commit/57bf0779bec61e2facd1679efc9bc5839e631d40 (and I likely should update Censor’s AI agreement, too). I personally use LLMs as "better search machine" in development sometimes, too, ensuring that all code I write is copyrightable by myself.
arthaud force-pushed main from affc84a9c5
Some checks failed
/ build-start (pull_request_target) Failing after 11s
/ release-prepare (pull_request_target) Has been skipped
/ checks (pull_request) Has been cancelled
/ merge-conditions (pull_request) Has been cancelled
to bce8016afa
Some checks failed
/ build-start (pull_request_target) Failing after 15s
/ checks (pull_request) Has been cancelled
/ merge-conditions (pull_request) Has been cancelled
2026年07月01日 05:22:16 +02:00
Compare
arthaud force-pushed main from bce8016afa
Some checks failed
/ build-start (pull_request_target) Failing after 15s
/ checks (pull_request) Has been cancelled
/ merge-conditions (pull_request) Has been cancelled
to bb045a109a
Some checks failed
/ build-start (pull_request_target) Failing after 9s
/ checks (pull_request) Has been cancelled
/ merge-conditions (pull_request) Has been cancelled
2026年07月01日 05:37:20 +02:00
Compare
Merge remote-tracking branch 'upstream/main'
Some checks failed
/ build-start (pull_request_target) Failing after 10s
/ checks (pull_request) Has been cancelled
/ merge-conditions (pull_request) Has been cancelled
d018410455
fix: accept additional arguments on actions
Some checks failed
/ checks (pull_request) Blocked by required conditions
/ merge-conditions (pull_request) Blocked by required conditions
/ build-start (pull_request_target) Failing after 9s
b1c33ab433
Author
First-time contributor
Copy link

Hi, thank you for your message.

  1. The goal is to implement the changes from #8, i.e. allow users to export redactions and re-import them. This improves QOL greatly because it means you can save your progress as you redact a huge document, and if you render it (by saving) and later realize you wanted to redact less —or any other issue —then you can recover your progress.
  2. I rewrote the LLM-generated parts of the code without using any AI on this PR. I will do the same on the other two shortly.
Hi, thank you for your message. 1. The goal is to implement the changes from #8, i.e. allow users to export redactions and re-import them. This improves QOL greatly because it means you can save your progress as you redact a huge document, and if you render it (by saving) and later realize you wanted to redact less —or any other issue —then you can recover your progress. 2. I rewrote the LLM-generated parts of the code without using any AI on this PR. I will do the same on the other two shortly.
Some checks failed
/ checks (pull_request) Blocked by required conditions
Required
Details
/ merge-conditions (pull_request) Blocked by required conditions
Required
Details
/ build-start (pull_request_target) Failing after 9s
Required
Details
This pull request doesn't have enough approvals yet. 0 of 1 approvals granted.
Some workflows are waiting to be reviewed.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u main:arthaud-main
git switch arthaud-main
Sign in to join this conversation.
No reviewers
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
censor/Censor!214
Reference in a new issue
censor/Censor
No description provided.
Delete branch "arthaud/Censor:main"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?