Skip to content

docs: Add Code of Conduct link and EU funding logo to README#87

Open
isanmaz wants to merge 1 commit intomainfrom
add-coc-and-eu-funding-logo
Open

docs: Add Code of Conduct link and EU funding logo to README#87
isanmaz wants to merge 1 commit intomainfrom
add-coc-and-eu-funding-logo

Conversation

@isanmaz
Copy link
Copy Markdown

@isanmaz isanmaz commented Mar 23, 2026

This PR adds a link to the Code of Conduct and the EU funding logo to the README.

Copilot AI review requested due to automatic review settings March 23, 2026 14:27
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the project README to improve community/project metadata visibility by adding a Code of Conduct link and displaying an EU funding logo.

Changes:

  • Adds an additional “Code of Conduct” section at the end of the README with a link to the org-wide CoC.
  • Adds a centered EU funding logo image to the README.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

README.md Outdated
Comment on lines +33 to +37

## Code of Conduct

To facilitate a nice environment for all, check out our [Code of Conduct](https://github.com/openkcm/.github/blob/main/CODE_OF_CONDUCT.md).

Copy link

Copilot AI Mar 23, 2026

Choose a reason for hiding this comment

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

README already contains a "## Code of Conduct" section earlier (lines ~25-28). This change adds a second "## Code of Conduct" heading and repeats the same link, which is redundant and can confuse readers. Suggest removing this new section and, if needed, extending the existing Code of Conduct section with any additional wording.

Suggested change
## Code of Conduct
To facilitate a nice environment for all, check out our [Code of Conduct](https://github.com/openkcm/.github/blob/main/CODE_OF_CONDUCT.md).

Copilot uses AI. Check for mistakes.
README.md Outdated
Comment on lines +34 to +36
## Code of Conduct

To facilitate a nice environment for all, check out our [Code of Conduct](https://github.com/openkcm/.github/blob/main/CODE_OF_CONDUCT.md).
Copy link

Copilot AI Mar 23, 2026

Choose a reason for hiding this comment

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

The EU funding logo is currently placed directly under the duplicated "Code of Conduct" section, which mixes unrelated content. Consider introducing a dedicated heading (e.g., "## Funding" / "## Acknowledgements") and a short caption/link describing the funding context, then place the logo there.

Suggested change
## Code of Conduct
To facilitate a nice environment for all, check out our [Code of Conduct](https://github.com/openkcm/.github/blob/main/CODE_OF_CONDUCT.md).
## Funding
This project receives funding support as indicated by the programme shown in the logo below.

Copilot uses AI. Check for mistakes.
README.md Outdated

Copyright (20xx-)20xx SAP SE or an SAP affiliate company and OpenKCM contributors. Please see our [LICENSE](LICENSE) for copyright and license information. Detailed information including third-party components and their licensing/copyright information is available [via the REUSE tool](https://api.reuse.software/info/github.com/openkcm/krypton).


Copy link

Copilot AI Mar 23, 2026

Choose a reason for hiding this comment

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

There are two consecutive blank lines inserted after the Licensing section (lines 32-33). Consider keeping spacing consistent with the rest of the README by using a single blank line between sections.

Suggested change

Copilot uses AI. Check for mistakes.
@jithinkunjachan jithinkunjachan force-pushed the add-coc-and-eu-funding-logo branch from 1211821 to 8f1644b Compare March 27, 2026 09:59
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.

4 participants