Skip to content

License update - #56

Open
Gautam8387 wants to merge 3 commits into
NygenAnalytics:masterfrom
Gautam8387:license
Open

Gautam8387 wants to merge 3 commits into
NygenAnalytics:masterfrom
Gautam8387:license

Conversation

@Gautam8387

Copy link
Copy Markdown
Member

Updated license to CC BY-NC-SA 4.0.

@qodo-code-review

Copy link
Copy Markdown

PR Summary by Qodo

Adopt CC BY-NC-SA 4.0 licensing

📝 Documentation ⚙️ Configuration changes 🕐 10-20 Minutes

Grey Divider

AI Description

• Replaces MIT licensing with CC BY-NC-SA 4.0 for code and documentation.
• Adds full license terms, copyright notices, and commercial licensing guidance.
• Aligns package metadata and README references with the new terms.
Diagram

graph TD
  D["DESCRIPTION"] --> L["LICENSE"] --> CC["CC BY-NC-SA 4.0"]
  R["README.md"] --> LM["LICENSE.md"] --> CC
Loading
High-Level Assessment

The following are alternative approaches to this PR:

1. Use a software-specific license
  • ➕ Can address source code distribution and modification more explicitly
  • ➕ May better align with established software licensing practices
  • ➖ Requires selecting or drafting terms matching the intended noncommercial restrictions
  • ➖ May not preserve the exact Creative Commons attribution and ShareAlike model
2. Keep one canonical full license copy
  • ➕ Avoids duplicating hundreds of lines across LICENSE and LICENSE.md
  • ➕ Reduces the risk of future legal-text drift
  • ➖ The Markdown license page would no longer be fully self-contained
  • ➖ Consumers may need to follow a link to read the complete terms

Recommendation: Confirm that applying CC BY-NC-SA 4.0 to software is the intended legal strategy, since Creative Commons generally recommends software-specific licenses for code. If retained, use LICENSE as the canonical full text and keep LICENSE.md focused on the summary, API terms, and a link to that canonical copy.

Files changed (4) +899 / -23

Documentation (3) +898 / -22
LICENSEReplace the MIT template with CC BY-NC-SA 4.0 terms +445/-2

Replace the MIT template with CC BY-NC-SA 4.0 terms

• Adds 2025 and 2026 copyright notices and the complete CC BY-NC-SA 4.0 International license text for the package code and documentation.

LICENSE

LICENSE.mdDocument noncommercial repository and API licensing +450/-18

Document noncommercial repository and API licensing

• Replaces the MIT terms with CC BY-NC-SA 4.0, adds commercial licensing contact guidance, corrects the generated-data license name, and embeds the full license text.

LICENSE.md

README.mdAdvertise the new license and commercial licensing path +3/-2

Advertise the new license and commercial licensing path

• Updates the license badge and repository license section to CC BY-NC-SA 4.0, including contact details for commercial licensing.

README.md

Other (1) +1 / -1
DESCRIPTIONPoint package metadata to the custom license file +1/-1

Point package metadata to the custom license file

• Removes the MIT declaration and configures the R package to derive its license solely from LICENSE.

DESCRIPTION

@qodo-code-review

qodo-code-review Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📜 Skill insights (0)

Grey Divider


Remediation recommended

1. Readers miss author attribution ✓ Resolved 🐞 Bug ≡ Correctness
Description
LICENSE adds the 2026 CyteTypeR authors as copyright holders, while the README-linked LICENSE.md
still identifies only Nygen Analytics AB. Because both files present themselves as the repository
license and the README routes users to the incomplete one, recipients receive inconsistent
attribution information.
Code

LICENSE[2]

+Copyright (c) 2026 CyteTypeR authors
Evidence
The package metadata selects LICENSE as its license file, and that file names both Nygen Analytics
AB and the 2026 CyteTypeR authors. The human-facing license document names only Nygen Analytics AB,
while the README directs users to that incomplete document.

DESCRIPTION[4-8]
LICENSE[1-7]
LICENSE.md[3-9]
README.md[147-150]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The canonical `LICENSE` names the 2026 CyteTypeR authors as copyright holders, but the README-linked `LICENSE.md` omits them, creating inconsistent attribution information.

## Fix Focus Areas
- LICENSE[1-6]
- LICENSE.md[3-9]
- README.md[147-150]

## Recommended Fix
Add the 2026 CyteTypeR authors copyright notice to `LICENSE.md` so it matches the canonical `LICENSE` file.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

Context sources
Review mode: ⚖️ Balanced: This is a licensing and package metadata contract change with potentially significant legal and distribution implications, so it warrants a careful single-pass review.

Grey Divider

Tip of the day
💡 Did you know, you can route each action level your way: inline, summary, both, or drop

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

Comment thread LICENSE
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.

1 participant