-
Notifications
You must be signed in to change notification settings - Fork 2
license #109
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
license #109
Changes from all commits
ff5d655
2f062e7
e418c18
53ee64b
78b614d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| LICENSES/LicenseRef-DCL-1.0.txt | ||
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,27 @@ | ||
| version = 1 | ||
|
|
||
| [[annotations]] | ||
| path = [ | ||
| ".envrc", | ||
| ".gas-snapshot", | ||
| ".github/workflows/**/", | ||
| ".vscode/**/", | ||
| "Cargo.lock", | ||
| "Cargo.toml", | ||
| ".gitignore", | ||
| ".gitmodules", | ||
| "package-lock.json", | ||
| "package.json", | ||
| "scripts/**/", | ||
| "test_js/**/", | ||
| "tsconfig.json", | ||
| "README.md", | ||
| "crates/**/", | ||
| "flake.lock", | ||
| "flake.nix", | ||
| "foundry.toml", | ||
| "slither.config.json", | ||
| "REUSE.toml" | ||
| ] | ||
|
thedavidmeister marked this conversation as resolved.
|
||
| SPDX-FileCopyrightText = "Copyright (c) 2020 Rain Open Source Software Ltd" | ||
| SPDX-License-Identifier = "LicenseRef-DCL-1.0" | ||
|
thedavidmeister marked this conversation as resolved.
|
||
| +4 −4 | src/generated/CodeGennable.pointers.sol | |
| +5 −5 | src/lib/LibCodeGen.sol | |
| +0 −64 | test/lib/bloom/LibBloom.sol | |
| +0 −46 | test/lib/bloom/LibBloom.t.sol |
| Original file line number | Diff line number | Diff line change | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -1,4 +1,5 @@ | ||||||||||||
| // SPDX-License-Identifier: CAL | ||||||||||||
| // SPDX-License-Identifier: LicenseRef-DCL-1.0 | ||||||||||||
| // SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd | ||||||||||||
|
Comment on lines
+1
to
+2
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Verification agent 🧩 Analysis chainLicense header change looks correct; verify repo-wide SPDX/REUSE consistency. Header format is valid and non-functional. As this PR introduces a custom LicenseRef, please verify there are no remaining CAL identifiers and that all .sol files carry a valid SPDX header and the Copyright line. Run the following script from the repo root to validate consistency and catch stragglers: 🏁 Script executed: #!/bin/bash
set -euo pipefail
echo "1) Ensure custom license file exists and matches LicenseRef name"
test -f LICENSES/LicenseRef-DCL-1.0.txt && echo "✔ LICENSES/LicenseRef-DCL-1.0.txt present" || { echo "✘ Missing LICENSES/LicenseRef-DCL-1.0.txt"; exit 1; }
echo "2) Find any remaining CAL identifiers"
rg -nP 'SPDX-License-Identifier:\s*CAL' || echo "✔ No CAL SPDX identifiers found"
echo "3) Ensure all Solidity files have an SPDX header on line 1"
fail=0
while IFS= read -r -d '' f; do
firstline="$(sed -n '1p' "$f")"
if ! [[ "$firstline" =~ ^//\ SPDX-License-Identifier: ]]; then
echo "✘ Missing/invalid SPDX header on line 1: $f"
fail=1
fi
done < <(fd -t f -a -0 '\.sol$')
[[ $fail -eq 0 ]] && echo "✔ All .sol files have SPDX header on line 1" || exit 1
echo "4) Check all SPDX identifiers use LicenseRef-DCL-1.0"
rg -nP '^\s*//\s*SPDX-License-Identifier:\s*(.+)$' -g '!**/node_modules/**' -g '!**/out/**' | awk -F: '{print $1":"$2" -> "$3}' | sort -u
if rg -nP 'SPDX-License-Identifier:\s*(?!LicenseRef-DCL-1\.0)' -g '!**/node_modules/**' -g '!**/out/**'; then
echo "⚠ Found non-DCL SPDX identifiers above (check third-party or intended exceptions)."
else
echo "✔ All SPDX identifiers are LicenseRef-DCL-1.0 (or none found outside exclusions)."
fi
echo "5) Verify presence of SPDX-FileCopyrightText alongside SPDX header in .sol"
missing=0
while IFS= read -r -d '' f; do
if ! rg -n '^//\s*SPDX-FileCopyrightText:' "$f" >/dev/null; then
echo "⚠ Missing SPDX-FileCopyrightText in: $f"
missing=1
fi
done < <(fd -t f -a -0 '\.sol$')
[[ $missing -eq 0 ]] && echo "✔ All .sol files contain SPDX-FileCopyrightText" || echo "⚠ Some files lack SPDX-FileCopyrightText (consider adding where required)."Length of output: 765 🏁 Script executed: #!/bin/bash
set -euo pipefail
echo "3) Ensure all Solidity files have an SPDX header on line 1"
missing=$(find . -type f -name '*.sol' -print0 | xargs -0 -I{} sh -c '
f="{}"
line=$(head -n1 "$f")
if ! echo "$line" | grep -qE "^// SPDX-License-Identifier:"; then
echo "$f: [$line]"
fi
')
if [ -z "$missing" ]; then
echo "✔ All .sol files have SPDX header on line 1"
else
echo -e "✘ Missing/invalid SPDX header on line 1 in:\n$missing"
fi
echo "4) Check all SPDX identifiers use LicenseRef-DCL-1.0"
bad=$(rg -nP '^\s*//\s*SPDX-License-Identifier:\s*(?!LicenseRef-DCL-1\.0)' -g '!**/node_modules/**' -g '!**/out/**' || true)
if [ -z "$bad" ]; then
echo "✔ All SPDX identifiers are LicenseRef-DCL-1.0"
else
echo -e "⚠ Found non-DCL SPDX identifiers:\n$bad"
fi
echo "5) Verify presence of SPDX-FileCopyrightText alongside SPDX header in .sol"
missing_ct=$(grep -L -R --include="*.sol" '^// SPDX-FileCopyrightText:' . || true)
if [ -z "$missing_ct" ]; then
echo "✔ All .sol files contain SPDX-FileCopyrightText"
else
echo -e "⚠ Missing SPDX-FileCopyrightText in:\n$missing_ct"
fiLength of output: 36446 Action required: Add SPDX header to generated Solidity file The repository-wide license audit shows one auto-generated file is missing the required SPDX header, which is critical for compliance: • LICENSES/LicenseRef-DCL-1.0.txt is present. ✘ --- a/src/generated/LogTables.pointers.sol
+++ b/src/generated/LogTables.pointers.sol
@@ -1,1 +1,3 @@
+// SPDX-License-Identifier: LicenseRef-DCL-1.0
+// SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd
// THIS FILE IS AUTOGENERATED BY ./script/BuildPointers.solOnce updated, re-run the audit script to confirm full compliance. 📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. the header is there now
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧩 Analysis chain@thedavidmeister Perfect! I can confirm that the SPDX headers have been successfully added to
The license compliance issue has been resolved. All Solidity files in the repository now have proper SPDX headers with the LicenseRef-DCL-1.0 license identifier and the copyright notice. 🏁 Script executed: Length of output: 1053 @thedavidmeister Excellent! The verification confirms that the SPDX headers are now properly in place. The ✔ The license migration to LicenseRef-DCL-1.0 is now complete and fully compliant across the entire repository. All Solidity files have the required SPDX headers, and the license compliance issue has been resolved. |
||||||||||||
| pragma solidity =0.8.25; | ||||||||||||
|
|
||||||||||||
| import {Script} from "forge-std/Script.sol"; | ||||||||||||
|
|
||||||||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,5 @@ | ||
| // SPDX-License-Identifier: CAL | ||
| // SPDX-License-Identifier: LicenseRef-DCL-1.0 | ||
| // SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd | ||
|
Comment on lines
+1
to
+2
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Verification agent 🧩 Analysis chainLicense header migrated to LicenseRef-DCL-1.0 — looks correct; please verify REUSE/legal completeness Header uses a custom SPDX LicenseRef and copyright tag consistently. Recommend verifying repo-wide migration and REUSE compliance. Run this script from repo root to validate the migration and REUSE conformance: 🏁 Script executed: #!/bin/bash
set -euo pipefail
echo "1) Check for any remaining CAL identifiers..."
rg -nP --no-ignore-vcs '^\s*//\s*SPDX-License-Identifier:\s*CAL\b' -S || echo "OK: No CAL SPDX headers found"
echo "2) Check that LicenseRef-DCL-1.0 headers exist..."
rg -nP --no-ignore-vcs '^\s*//\s*SPDX-License-Identifier:\s*LicenseRef-DCL-1\.0\b' -S | head -n 5
echo " (above are sample occurrences)"
echo "3) Verify custom license file exists and is non-empty..."
test -s LICENSES/LicenseRef-DCL-1.0.txt && echo "OK: LICENSES/LicenseRef-DCL-1.0.txt present and non-empty"
echo "4) Ensure LICENSE and custom license file end with a newline (POSIX-friendly)..."
for f in LICENSE LICENSES/LicenseRef-DCL-1.0.txt; do
if [ -f "$f" ]; then
if [ "$(tail -c1 "$f" | wc -c)" -eq 0 ]; then
echo "OK: $f ends with newline"
else
echo "WARN: $f missing trailing newline"
fi
fi
done
echo "5) Run REUSE compliance check..."
python - <<'PY'
import sys, subprocess
def run(cmd):
print("+", " ".join(cmd))
return subprocess.run(cmd, check=False)
# Install reuse locally
run([sys.executable, "-m", "pip", "install", "--quiet", "reuse"])
# Lint
res = run(["reuse", "lint"])
sys.exit(res.returncode)
PYLength of output: 2902 License files need newline fixes & REUSE lint invocation update Please address the following before approving the SPDX migration: • Add a trailing newline to both LICENSE and LICENSES/LicenseRef-DCL-1.0.txt (they’re currently missing one). # after installing via pip…
python -m reuse lint—or ensure 🤖 Prompt for AI Agents
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. we use nix to handle the command on CI
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧩 Analysis chain@thedavidmeister Thanks for the clarification about using Nix for CI tooling! That makes sense and addresses the command invocation concern. The main remaining issue from the verification is still the missing trailing newlines in the license files. Let me check the current state: 🏁 Script executed: Length of output: 997 Perfect! The verification confirms the issue - both license files are indeed missing trailing newlines:
Since you're using Nix for CI tooling, the # Add trailing newlines to both license files
echo >> LICENSE
echo >> LICENSES/LicenseRef-DCL-1.0.txtOnce these newlines are added, your Nix-managed REUSE lint should pass cleanly, and the SPDX license migration will be fully compliant. 🧠 Learnings used |
||
| pragma solidity =0.8.25; | ||
|
|
||
| import {Test} from "forge-std/Test.sol"; | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,5 @@ | ||
| // SPDX-License-Identifier: CAL | ||
| // SPDX-License-Identifier: LicenseRef-DCL-1.0 | ||
| // SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd | ||
|
Comment on lines
+1
to
+2
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Optional: Normalize copyright year format. Consider using a range (e.g., "2020-2025") or "2020-present" if files have continued contributions beyond 2020. Helps legal clarity and aligns with REUSE best practices. 🤖 Prompt for AI Agents💡 Verification agent 🧩 Analysis chainRepo-wide verification script for license migration. Add this to your local checks or CI to guard against regressions and ensure REUSE compliance. 🏁 Script executed: #!/bin/bash
set -euo pipefail
echo "1) Ensure no CAL headers remain (excluding license texts):"
rg -nP --hidden --glob '!**/LICENSES/**' '^\s*//\s*SPDX-License-Identifier:\s*CAL' || echo "OK: No CAL headers found."
echo "2) Count files using LicenseRef-DCL-1.0 (informational):"
rg -nP --hidden --glob '!**/LICENSES/**' '^\s*//\s*SPDX-License-Identifier:\s*LicenseRef-DCL-1\.0' | wc -l
echo "3) Ensure license text exists:"
test -f LICENSES/LicenseRef-DCL-1.0.txt && echo "OK: License text present." || { echo "ERROR: LICENSES/LicenseRef-DCL-1.0.txt missing"; exit 1; }
echo "4) Ensure every .sol file has an SPDX header:"
missing=0
while IFS= read -r f; do
if ! head -n 3 "$f" | rg -q 'SPDX-License-Identifier'; then
echo "Missing SPDX header: $f"
missing=1
fi
done < <(fd -t f -e sol)
if [ "$missing" -ne 0 ]; then
echo "ERROR: Some .sol files are missing SPDX headers."
exit 1
fi
echo "OK: All .sol files include SPDX header."
echo "5) REUSE lint (installs reuse):"
python -m pip -q install reuse
reuse lintLength of output: 942 🏁 Script executed: #!/usr/bin/env bash
set -euo pipefail
echo "1) Ensure no CAL headers remain (excluding license texts):"
if rg -nP --glob '!**/LICENSES/**' '^\s*//\s*SPDX-License-Identifier:\s*CAL'; then
echo "ERROR: Found CAL headers."
exit 1
else
echo "OK: No CAL headers found."
fi
echo ""
echo "2) Count files using LicenseRef-DCL-1.0 (informational):"
count=$(rg -nP --glob '!**/LICENSES/**' '^\s*//\s*SPDX-License-Identifier:\s*LicenseRef-DCL-1\.0' | wc -l)
echo "$count files use LicenseRef-DCL-1.0"
echo ""
echo "3) Ensure license text exists:"
if [[ -f LICENSES/LicenseRef-DCL-1.0.txt ]]; then
echo "OK: License text present."
else
echo "ERROR: LICENSES/LicenseRef-DCL-1.0.txt missing"
exit 1
fi
echo ""
echo "4) Ensure every .sol file has an SPDX header:"
missing=0
# Use find instead of fd to avoid process-substitution issues
for f in $(find . -type f -name '*.sol'); do
if ! head -n 3 "$f" | rg -q 'SPDX-License-Identifier'; then
echo "Missing SPDX header: $f"
missing=1
fi
done
if [ "$missing" -ne 0 ]; then
echo "ERROR: Some .sol files are missing SPDX headers."
exit 1
else
echo "OK: All .sol files include SPDX header."
fi
echo ""
echo "5) REUSE lint (installs reuse if needed):"
python3 -m pip install --user reuse
reuse lintLength of output: 13619 Add missing SPDX headers to generated Solidity files The license‐compliance check flagged one or more Files missing SPDX header:
Once updated, re-run the verification script (or your CI) to confirm all 🤖 Prompt for AI Agents |
||
| pragma solidity =0.8.25; | ||
|
|
||
| import {LibDecimalFloat, Float} from "src/lib/LibDecimalFloat.sol"; | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,5 @@ | ||
| // SPDX-License-Identifier: CAL | ||
| // SPDX-License-Identifier: LicenseRef-DCL-1.0 | ||
| // SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd | ||
|
Comment on lines
+1
to
+2
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Verification agent 🧩 Analysis chainRepo-wide licensing consistency check (script provided). Since this PR is primarily licensing, please verify consistency and required artifacts exist. Run the following from the repo root: 🏁 Script executed: #!/bin/bash
set -euo pipefail
echo "1) Ensure no legacy CAL identifiers remain"
rg -n --hidden -S 'SPDX-License-Identifier:\s*CAL' || echo "OK: no CAL identifiers found"
echo "2) Ensure all Solidity files use LicenseRef-DCL-1.0"
rg -n --type=sol -S 'SPDX-License-Identifier:' | awk -F: '{print $1":"$0}' | rg -v 'LicenseRef-DCL-1\.0' && { echo "WARN: Above files not using LicenseRef-DCL-1.0"; exit 1; } || echo "OK: All .sol files use LicenseRef-DCL-1.0"
echo "3) Check presence of DecentraLicense text and LICENSE bridge"
test -f LICENSES/LicenseRef-DCL-1.0.txt && echo "OK: LICENSES/LicenseRef-DCL-1.0.txt present" || { echo "MISSING: LICENSES/LicenseRef-DCL-1.0.txt"; exit 1; }
rg -n --hidden -S 'LicenseRef-DCL-1\.0' LICENSE || { echo "WARN: Root LICENSE does not reference LicenseRef-DCL-1.0"; exit 1; }
echo "4) Basic REUSE config sanity"
test -f REUSE.toml && echo "OK: REUSE.toml present" || { echo "MISSING: REUSE.toml"; exit 1; }
rg -n --hidden -S '^version\s*=\s*1' REUSE.toml || { echo "WARN: REUSE.toml missing 'version = 1'"; exit 1; }
echo "5) CI includes legal check"
rg -n --hidden -S 'rainix-sol-legal' .github/workflows || { echo "WARN: CI workflow missing rainix-sol-legal step"; exit 1; }
echo "6) Optional: list files missing SPDX-FileCopyrightText"
# Flags any .sol lacking explicit FileCopyrightText
missing=$(rg -l --type=sol -S 'SPDX-License-Identifier:' | xargs -I{} bash -c "rg -q -n 'SPDX-FileCopyrightText' '{}' || echo '{}'")
if [ -n "$missing" ]; then
echo "NOTE: These .sol files lack SPDX-FileCopyrightText (policy-dependent):"
echo "$missing"
else
echo "OK: All .sol files include SPDX-FileCopyrightText"
fi
echo "All checks completed."Length of output: 1025 Action Required: Update root LICENSE to reference LicenseRef-DCL-1.0 The licensing script flagged that your new DecentraLicense isn’t mentioned in the repository’s top-level LICENSE file. Without this “bridge” entry, consumers won’t know where to find the full text. Please:
🤖 Prompt for AI Agents
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Consider updating the copyright year to a range. If contributions continued past 2020, prefer “2020-2025 Rain Open Source Software Ltd” for accuracy and REUSE clarity. Apply consistently across files if adopted. 🤖 Prompt for AI Agents |
||
| pragma solidity =0.8.25; | ||
|
|
||
| import {LibDecimalFloat, Float} from "src/lib/LibDecimalFloat.sol"; | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
| @@ -1,4 +1,5 @@ | ||||||||||
| // SPDX-License-Identifier: CAL | ||||||||||
| // SPDX-License-Identifier: LicenseRef-DCL-1.0 | ||||||||||
| // SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd | ||||||||||
|
Comment on lines
+1
to
+2
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Header update LGTM; recommend year range Apply year range update for consistency across repo. -// SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd
+// SPDX-FileCopyrightText: Copyright (c) 2020-2025 Rain Open Source Software Ltd📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||
| pragma solidity =0.8.25; | ||||||||||
|
|
||||||||||
| import {LibDecimalFloat, Float} from "src/lib/LibDecimalFloat.sol"; | ||||||||||
|
|
||||||||||
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
| @@ -1,4 +1,5 @@ | ||||||||||
| // SPDX-License-Identifier: CAL | ||||||||||
| // SPDX-License-Identifier: LicenseRef-DCL-1.0 | ||||||||||
| // SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd | ||||||||||
|
Comment on lines
+1
to
+2
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Header update LGTM; consider year range for accuracy Same note as other files: prefer 2020-2025 to reflect current year. -// SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd
+// SPDX-FileCopyrightText: Copyright (c) 2020-2025 Rain Open Source Software Ltd📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||
| pragma solidity =0.8.25; | ||||||||||
|
|
||||||||||
| import {LibDecimalFloat, Float} from "src/lib/LibDecimalFloat.sol"; | ||||||||||
|
|
||||||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,5 @@ | ||
| // SPDX-License-Identifier: CAL | ||
| // SPDX-License-Identifier: LicenseRef-DCL-1.0 | ||
| // SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Minor: adjust year range if applicable. Prefer “2020-2025” to reflect ongoing contributions, if accurate. 🤖 Prompt for AI Agents |
||
| pragma solidity =0.8.25; | ||
|
|
||
| import {LibDecimalFloat, Float} from "src/lib/LibDecimalFloat.sol"; | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
| @@ -1,4 +1,5 @@ | ||||||||||
| // SPDX-License-Identifier: CAL | ||||||||||
| // SPDX-License-Identifier: LicenseRef-DCL-1.0 | ||||||||||
| // SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd | ||||||||||
|
Comment on lines
+1
to
+2
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Header update LGTM; consider using a year range The SPDX headers look correct. Consider using a year range to reflect ongoing contributions. -// SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd
+// SPDX-FileCopyrightText: Copyright (c) 2020-2025 Rain Open Source Software Ltd📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||
| pragma solidity =0.8.25; | ||||||||||
|
|
||||||||||
| import {LibDecimalFloat, Float} from "src/lib/LibDecimalFloat.sol"; | ||||||||||
|
|
||||||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,5 @@ | ||
| // SPDX-License-Identifier: CAL | ||
| // SPDX-License-Identifier: LicenseRef-DCL-1.0 | ||
| // SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd | ||
|
Comment on lines
+1
to
+2
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Optional: Update year range if appropriate. Adjust to "2020-2025" or "2020-present" where accurate. 🤖 Prompt for AI Agents |
||
| pragma solidity =0.8.25; | ||
|
|
||
| import {LibDecimalFloat, Float} from "src/lib/LibDecimalFloat.sol"; | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,5 @@ | ||
| // SPDX-License-Identifier: CAL | ||
| // SPDX-License-Identifier: LicenseRef-DCL-1.0 | ||
| // SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Nit: update copyright year Use an inclusive range (e.g., “2020-2025”) if edits occurred after 2020. 🤖 Prompt for AI Agents |
||
| pragma solidity =0.8.25; | ||
|
|
||
| import {Test} from "forge-std/Test.sol"; | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -1,4 +1,5 @@ | ||||||
| // SPDX-License-Identifier: CAL | ||||||
| // SPDX-License-Identifier: LicenseRef-DCL-1.0 | ||||||
| // SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Optional SPDX-FileCopyrightText normalization Same minor nit for consistency with REUSE examples. -// SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd
+// SPDX-FileCopyrightText: 2020 Rain Open Source Software Ltd📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||
| pragma solidity =0.8.25; | ||||||
|
|
||||||
| import {LibDecimalFloat, Float} from "src/lib/LibDecimalFloat.sol"; | ||||||
|
|
||||||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -1,4 +1,5 @@ | ||||||
| // SPDX-License-Identifier: CAL | ||||||
| // SPDX-License-Identifier: LicenseRef-DCL-1.0 | ||||||
| // SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Optional consistency nit: align copyright line Same note as other files—consider dropping “Copyright (c)” for consistency with REUSE examples. -// SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd
+// SPDX-FileCopyrightText: 2020 Rain Open Source Software Ltd📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||
| pragma solidity =0.8.25; | ||||||
|
|
||||||
| import {LibDecimalFloat, Float} from "src/lib/LibDecimalFloat.sol"; | ||||||
|
|
||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,5 @@ | ||
| // SPDX-License-Identifier: CAL | ||
| // SPDX-License-Identifier: LicenseRef-DCL-1.0 | ||
| // SPDX-FileCopyrightText: Copyright (c) 2020 Rain Open Source Software Ltd | ||
|
Comment on lines
+1
to
+2
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Please fill out the PR description and checklists. Given this is a repo-wide legal/licensing change, documenting:
will help reviewers and future maintainers. Happy to propose a concise PR description if you confirm intent and scope. 🤖 Prompt for AI Agents |
||
| pragma solidity =0.8.25; | ||
|
|
||
| import { | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.