Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update from Working-model: bc3df0f #135

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

CX-Machine-User
Copy link

@CX-Machine-User CX-Machine-User commented Oct 21, 2024

This PR updates the branch with the latest changes from Working-model. Commit ID: bc3df0f

These are the last changes from working model. Would like to merge them, before we execute the versioning plug-in.

@ther3sa and @werner-roman you are the only once who are left for review :)

Copy link
Contributor

Choose a reason for hiding this comment

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

Not readable in dark mode - needs background

@@ -1,16 +1,15 @@
---
sidebar_position: 5
title: Catena-X office
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should keep the h1 heading - as this is used by all the other documents as well - at leas I don't see a benefit in using title?

Copy link
Contributor

Choose a reason for hiding this comment

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

we are usually using title instead of h1

Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a benefit? otherwise i think we should keep it consistent across all documents?

@@ -33,7 +33,7 @@ There are two main organizations in GitHub:

:::note

- for the open-source part (reference implementations and Kits) GitHub is mandatory. Its used for planning, issue tracking, discussions, development, reviews and release etc.
- for the open-source part (reference implementations and KITs) GitHub is mandatory. Its used for planning, issue tracking, discussions, development, reviews and release etc.
- for the standardization part, GitHub is also set as mandatory
- for Committee work and Expert Groups the Association provides private repositories within the *catenax-eV* GitHub organization
Copy link
Contributor

Choose a reason for hiding this comment

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

link ti GitHub org?

Copy link
Contributor

Choose a reason for hiding this comment

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

Not readable in dark mode - needs background

Copy link
Contributor

Choose a reason for hiding this comment

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

Not readable in dark mode - needs background

@jSchuetz88
Copy link
Member

jSchuetz88 commented Nov 14, 2024

Why is this PR empty? is this still up-to-date? Can we close this PR?

@stephanbcbauer
Copy link
Member

since this was never merged, should we delete this PR and create a new one? -> i think the comments by @werner-roman are still valid ;) @Grand-Thibault

@stephanbcbauer
Copy link
Member

@Grand-Thibault would like to delete this PR, since the changes are old, but I still don't want to lose the comments by @werner-roman. @Grand-Thibault, can you please fix it in the working-model?

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.

6 participants