Skip to content

template: initial template - #1

Merged
AlexanderLanin merged 2 commits into
eclipse-score:mainfrom
qorix-group:dcalavrezo_new_template
Mar 11, 2025
Merged

AlexanderLanin merged 2 commits into
eclipse-score:mainfrom
qorix-group:dcalavrezo_new_template

Conversation

@dcalavrezo-qorix

Copy link
Copy Markdown
Contributor

Initial template to be used

Fixes: #511

Initial template to be used

Fixes: #511

Signed-off-by: Dan Calavrezo <dan.calavrezo.ext@qorix.ai>
Comment thread .github/CODEOWNERS
Comment thread .vscode/settings.json
@@ -0,0 +1,101 @@
{

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.

Soo much duplication... maybe we do need to consider how to sync this with the score repo?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

same goes for docs

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think that was the plan 'after' anyway? To have most of the pieces here in their own repo so we can just pull them in, or did I missremember?

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.

modules will use this template repo. But score does not?! So we have duplication between the template repo and score. I'm not sure whether score should use this template as well?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Is score a module? Then the answer is no

Comment thread MODULE.bazel
Comment thread README.md Outdated
Signed-off-by: Dan Calavrezo <dan.calavrezo.ext@qorix.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants