Skip to content

Add Claude setup#168

Open
hcarreras wants to merge 2 commits into
mainfrom
add-claude-setup
Open

Add Claude setup#168
hcarreras wants to merge 2 commits into
mainfrom
add-claude-setup

Conversation

@hcarreras
Copy link
Copy Markdown
Contributor

It needs an organization, so instructions are needed.

Copy link
Copy Markdown
Contributor

@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 updates the client infrastructure setup runbook and access permissions documentation to include guidelines for setting up Claude (platform.claude.com) organizations and billing. A review comment points out that a local settings file .claude/settings.local.json was accidentally committed and should be removed, with .claude/ added to .gitignore.

I am having trouble creating individual review comments. Click here to see my feedback.

.claude/settings.local.json (1-7)

medium

It appears that .claude/settings.local.json was accidentally committed.

This file contains local permissions for a client-specific MCP server (brewale-abtion-mcp), which is unrelated to this company handbook repository. Local settings files generated by Claude Code (like *.local.json) are developer-specific and should not be committed to version control.

Recommendation

  1. Remove this file from the repository.
  2. Add .claude/ (or at least *.local.json) to the repository's .gitignore file to prevent accidental commits in the future.

It needs an organization, so instructions are needed.
Copy link
Copy Markdown
Member

@runephilosof-abtion runephilosof-abtion left a comment

Choose a reason for hiding this comment

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

This is only file name that starts with upper case.
That should be fixed, when the file is being renamed.


This applies only to `platform.claude.com` (the Anthropic Console used for API keys and agents). The Claude Code / claude.ai user subscription for day-to-day coding is a separate matter, included in our options and description and billed to the client at our normal pricing.

- Every client that needs API keys or agents has their own Claude organization, named after the team's google group. E.g. `danish-shipping`.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I prefer to avoid mentioning concrete clients in this public repo. However, I can see that this has been done before in this file 🤷

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