Skip to content

Prepare repository for open-sourcing#22

Merged
fank merged 2 commits intomainfrom
claude/open-source-prep
Feb 12, 2026
Merged

Prepare repository for open-sourcing#22
fank merged 2 commits intomainfrom
claude/open-source-prep

Conversation

@Hinne1
Copy link
Contributor

@Hinne1 Hinne1 commented Feb 9, 2026

Summary

  • Replace internal project key (NX) with generic PROJ in AGENTS.md examples
  • Replace company-specific custom field name with generic placeholder in AGENTS.md
  • Remove license_go.yml workflow (references private enthus-appdev/actions repo)
  • Add .claude/ to .gitignore
  • Use HTTPS instead of SSH for install.sh clone fallback (public accessibility)
  • Update LICENSE copyright holder to enthus GmbH

Context

Preparing this repository to be made public.

Test plan

  • Verify make build still works
  • Verify install.sh works without SSH keys configured
  • Review AGENTS.md for any remaining internal references

@gemini-code-assist
Copy link

Summary of Changes

Hello @Hinne1, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request prepares the repository for open-sourcing by sanitizing internal references, enhancing the installation process for public use, and updating legal documentation. The changes ensure that the repository is generic, accessible, and compliant for a wider audience.

Highlights

  • Documentation Updates: Replaced internal project key 'NX' with a generic 'PROJ' in AGENTS.md examples and updated a company-specific custom field name to a generic 'Custom Field'.
  • Repository Configuration: Added .claude/ to the .gitignore file to exclude specific development artifacts.
  • Installation Script Enhancement: Modified install.sh to use HTTPS for Git cloning as a fallback, improving public accessibility by removing the dependency on SSH keys.
  • Legal Information Update: Updated the copyright holder in the LICENSE file from 'enthus-appdev' to 'enthus GmbH'.
Changelog
  • .gitignore
    • Added .claude/ to the list of ignored files.
  • AGENTS.md
    • Replaced all instances of the internal project key NX with PROJ in various atl command examples.
    • Changed the custom field example from 'Technischer Hinweis' to 'Custom Field'.
  • LICENSE
    • Updated the copyright holder from 'enthus-appdev' to 'enthus GmbH'.
  • install.sh
    • Modified the fallback git clone command to use an HTTPS URL (https://github.com/${REPO}.git) instead of an SSH URL (git@github.com:${REPO}.git).
Ignored Files
  • Ignored by pattern: .github/workflows/** (1)
    • .github/workflows/license_go.yml
Activity
  • No human activity (comments, reviews, or progress updates) has been recorded on this pull request since its creation.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request does a good job of preparing the repository for open-sourcing. The changes, such as replacing internal project keys in AGENTS.md, updating the LICENSE, and switching to HTTPS for cloning in install.sh, are all necessary and well-executed steps. I've left one comment regarding a minor follow-up to the change in install.sh to improve user experience by making an error message consistent with the code change. Overall, the changes are solid and move the project in the right direction for a public release.

@Hinne1 Hinne1 force-pushed the claude/open-source-prep branch 2 times, most recently from ba0cbe1 to a9a54fb Compare February 12, 2026 11:55
Copy link
Member

@fank fank left a comment

Choose a reason for hiding this comment

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

Review: Prepare repository for open-sourcing

The PR covers the critical items well — private CI action removed, internal project keys scrubbed from AGENTS.md, SSH→HTTPS fallback fixed, and LICENSE updated. Good work.

However, a few community-standard files are missing that should be added before going public:

Requested changes

  1. Add CONTRIBUTING.md — Guide for external contributors: how to report bugs, submit PRs, development setup (make build, make test, make lint), and code style expectations.

  2. Add SECURITY.md — Instructions for reporting security vulnerabilities privately (e.g., via email or GitHub Security Advisories) instead of public issues.

  3. Add CODE_OF_CONDUCT.md — Community standard for public repos. The Contributor Covenant is a common choice.

These are expected by the open source community and signal that the project is ready for external participation.

@Hinne1
Copy link
Contributor Author

Hinne1 commented Feb 12, 2026

Thanks for the review. Regarding the community-standard files:

  1. CONTRIBUTING.md — Will add with dev setup (go build, make test, make lint), PR process, and code style notes.
  2. SECURITY.md — Will add with GitHub Security Advisories as the preferred channel.
  3. CODE_OF_CONDUCT.md — Will add Contributor Covenant.

I'll push these in a follow-up commit.

- Replace internal project key references with generic placeholders in AGENTS.md
- Remove license validator workflow (references private org action)
- Add .claude/ to .gitignore
- Use HTTPS instead of SSH for install.sh clone fallback
- Update LICENSE copyright holder to enthus GmbH
@fank
Copy link
Member

fank commented Feb 12, 2026

Checked again — no new commits since the review. The three requested files (CONTRIBUTING.md, SECURITY.md, CODE_OF_CONDUCT.md) are still missing.

Add CONTRIBUTING.md, SECURITY.md, and CODE_OF_CONDUCT.md as requested
in PR review.
@Hinne1 Hinne1 force-pushed the claude/open-source-prep branch from a9a54fb to edac47c Compare February 12, 2026 15:13
Copy link
Member

@fank fank left a comment

Choose a reason for hiding this comment

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

All requested changes addressed. LGTM.

@fank fank merged commit 235ecca into main Feb 12, 2026
8 checks passed
@fank fank deleted the claude/open-source-prep branch February 12, 2026 15:25
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