Skip to content

docs: add GitHub Copilot CLI installation instructions#215

Open
GeekTrainer wants to merge 1 commit into
UI5:mainfrom
GeekTrainer:main
Open

docs: add GitHub Copilot CLI installation instructions#215
GeekTrainer wants to merge 1 commit into
UI5:mainfrom
GeekTrainer:main

Conversation

@GeekTrainer

Copy link
Copy Markdown

Adds GitHub Copilot CLI installation instructions to the README under the Specific MCP Clients section.

Changes

  • Added "GitHub Copilot CLI" section after Cline
  • Includes interactive setup (/mcp add) and manual configuration reference
  • Links to the standard configuration section to avoid duplication
  • Links to official Copilot CLI documentation

Testing

  • Verified markdown renders correctly
  • Internal link to standard configuration section is valid

@cla-assistant

cla-assistant Bot commented Jan 30, 2026

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@GeekTrainer

Copy link
Copy Markdown
Author

Question - would this fall under a small code exception since it's a small docs update, or will you need a CLA to be signed?

@RandomByte RandomByte left a comment

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.

Thanks for the contribution.

It would be great if you could sign the CLA. It's possible that there is a pre-existing agreement between Microsoft and SAP regarding code contributions, you might need to consult your Open Source Program Office (or similar) for any questions regarding this.

Comment thread README.md
Use the Copilot CLI to interactively add the MCP server:

```bash
/mcp add

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.

Does the user also need to specify the name of the UI5 MCP server here?

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.

@flovogt

flovogt commented May 19, 2026

Copy link
Copy Markdown
Member

@GeekTrainer Looking forward to incorporate your contribution into the repository. Could you please have another look at the comments?

@GeekTrainer

Copy link
Copy Markdown
Author

@flovogt Sorry for the long delay here! It doesn't look like we already have a CLA agreement, and the process can be a bit complex. I am wondering if there's an exception available as it's a small docs update. If not I can work on pursuing the CLA signature.

@flovogt

flovogt commented May 20, 2026

Copy link
Copy Markdown
Member

I don't mind to get the contribution done be our team. For me its more about #215 (comment)

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.

3 participants