Skip to content

Merge updates from constructorfabric into main#7

Merged
claudedigon merged 4 commits into
constructorfabric:mainfrom
vzhuman:main
Jun 22, 2026
Merged

Merge updates from constructorfabric into main#7
claudedigon merged 4 commits into
constructorfabric:mainfrom
vzhuman:main

Conversation

@vzhuman

@vzhuman vzhuman commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

What changed

Another example of website changes preview - removed copyright word in website footer

Pages to review

  • index.html
  • foundation.html
  • elements.html
  • learn.html
  • participate.html
  • privacy-policy.html
  • 404.html
  • styles.css
  • partials.jsx
  • assets/
  • Other:

Preview

A preview URL will be posted automatically by GitHub Actions after CI succeeds. The format is:

https://constructorfabric.github.io/website-previews/pr-<NUMBER>/

If you do not see a preview URL on this PR within a few minutes of pushing, see docs/pr-preview-process.md for troubleshooting.

Screenshots

Required for visual changes. Optional for text-only changes.

Desktop:

Screenshot 2026-06-22 at 1 47 05 PM

Mobile:

Screenshot 2026-06-22 at 1 47 20 PM

Checklist

  • I checked the site locally (Live Preview extension or python3 -m http.server)
  • I listed every affected page above
  • I added screenshots for visual changes
  • I avoided unrelated formatting changes
  • I followed the banned-words and no-emojis conventions in CONTRIBUTING.md

Summary by CodeRabbit

  • Chores
    • Updated Developer Certificate of Origin configuration to support remediation commit options.

@constructorfabric

Copy link
Copy Markdown

Preview deployment is ready:

https://constructorfabric.github.io/website-previews/pr-7/

This preview is built from the latest commit on this PR. It is regenerated automatically on every push. Old previews are cleaned up after 30 days.

@constructorfabric constructorfabric deleted a comment from coderabbitai Bot Jun 22, 2026
@constructorfabric

Copy link
Copy Markdown

Preview deployment is ready:

https://constructorfabric.github.io/website-previews/pr-7/

This preview is built from the latest commit on this PR. It is regenerated automatically on every push. Old previews are cleaned up after 30 days.

@coderabbitai

coderabbitai Bot commented Jun 22, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The .github/dco.yml configuration file is updated to add an allowRemediationCommits block with both individual and thirdParty set to true, enabling remediation commits for individual contributors and third-party contributions.

Changes

DCO Configuration Update

Layer / File(s) Summary
Enable DCO remediation commits
.github/dco.yml
Adds allowRemediationCommits with individual: true and thirdParty: true to permit remediation commits from both individual and third-party contributors.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A rabbit hops by with a tweak so small,
A YAML line added, that's nearly all.
Remediation commits, now allowed to roam,
For third-party friends and contributors at home.
🐇 The DCO gate swings open with a gentle click!

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title refers to merging updates from constructorfabric into main, but the actual change is only a DCO configuration update in .github/dco.yml, unrelated to website changes described in the PR description. Update the title to accurately reflect the primary change, such as 'Update DCO configuration to enable remediation commits' or 'Enable allowRemediationCommits in DCO configuration'.
Description check ⚠️ Warning The PR description describes website footer changes and includes screenshots, but the actual change is a configuration file update unrelated to website content. The description and actual changes are mismatched. Update the description to accurately reflect the DCO configuration change, or verify that the correct changeset has been included in this PR.
✅ Passed checks (3 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
.github/dco.yml (1)

1-3: 🧹 Nitpick | 🔵 Trivial

DCO remediation commits enable lenient sign-off enforcement.

The configuration enables both individual and third-party contributors to add DCO sign-offs in subsequent "remediation" commits, rather than requiring proper sign-offs on the original contribution. This is a policy decision that relaxes the strictness of DCO enforcement.

Confirm that this aligns with your project's contributor governance and DCO standards.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.github/dco.yml around lines 1 - 3, Review your project's contributor
governance and DCO (Developer Certificate of Origin) standards to determine the
appropriate level of sign-off enforcement needed. Based on your policy
requirements, adjust the allowRemediationCommits configuration in the DCO
settings by modifying the individual and thirdParty boolean flags to match your
project's standards. If your project requires stricter sign-off enforcement on
original contributions, consider setting these values to false or adjusting them
according to your governance policy.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In @.github/dco.yml:
- Around line 1-3: Review your project's contributor governance and DCO
(Developer Certificate of Origin) standards to determine the appropriate level
of sign-off enforcement needed. Based on your policy requirements, adjust the
allowRemediationCommits configuration in the DCO settings by modifying the
individual and thirdParty boolean flags to match your project's standards. If
your project requires stricter sign-off enforcement on original contributions,
consider setting these values to false or adjusting them according to your
governance policy.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c255b218-4227-4397-a75c-d667e9fbf5e9

📥 Commits

Reviewing files that changed from the base of the PR and between 535a452 and f12677a.

📒 Files selected for processing (1)
  • .github/dco.yml

@claudedigon claudedigon merged commit fdbd3a5 into constructorfabric:main Jun 22, 2026
2 of 3 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