Skip to content

doc: initial config mgt process #845

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.

Sign up for GitHub

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

Jump to bottom
Merged
aschemmel-tech merged 1 commit into from
Apr 10, 2025
Merged

doc: initial config mgt process #845

aschemmel-tech merged 1 commit into from
Apr 10, 2025

Conversation

@aschemmel-tech
Copy link
Contributor

@aschemmel-tech aschemmel-tech commented Apr 2, 2025

Ref: solves #538

@github-actions
Copy link

github-actions bot commented Apr 2, 2025

The created documentation from the pull request is available at: docu-html

@aschemmel-tech aschemmel-tech marked this pull request as ready for review April 2, 2025 13:57
@aschemmel-tech aschemmel-tech requested a review from masc2023 April 2, 2025 13:57
masc2023
masc2023 requested changes Apr 3, 2025
View reviewed changes
Copy link
Contributor

@masc2023 masc2023 left a comment

Choose a reason for hiding this comment

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

May consider to merge that with Release Management and have it as one PR?

@aschemmel-tech aschemmel-tech force-pushed the aschemmel-tech-config-mgt-init branch from 3b6c899 to 8aec57d Compare April 7, 2025 15:06
@github-actions
Copy link

github-actions bot commented Apr 7, 2025 •
edited
Loading

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //:license-check

Status: ✅ Passed

Click to expand output 
[License Check Output]
2025/04/10 11:35:42 Downloading https://releases.bazel.build/7.4.0/release/bazel-7.4.0-linux-x86_64...
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: 
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //:license-check (89 packages loaded, 10 targets configured)

Analyzing: target //:license-check (99 packages loaded, 10 targets configured)

Analyzing: target //:license-check (131 packages loaded, 824 targets configured)

Analyzing: target //:license-check (133 packages loaded, 1607 targets configured)

Analyzing: target //:license-check (145 packages loaded, 2465 targets configured)

Analyzing: target //:license-check (145 packages loaded, 2465 targets configured)

Analyzing: target //:license-check (148 packages loaded, 4487 targets configured)

INFO: Analyzed target //:license-check (150 packages loaded, 4737 targets configured).
[7 / 13] Creating runfiles tree bazel-out/k8-opt-exec-ST-d57f47055a04/bin/external/dash_license_checker~/tool/formatters/dash_format_converter.runfiles [for tool]; 0s local ... (2 actions, 1 running)
[11 / 13] [Prepa] JavaToolchainCompileBootClasspath external/rules_java~/toolchains/platformclasspath.jar
INFO: Found 1 target...
Target //:license.check.license_check up-to-date:
  bazel-bin/license.check.license_check
  bazel-bin/license.check.license_check.jar
INFO: Elapsed time: 17.573s, Critical Path: 2.18s
INFO: 13 processes: 9 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 13 total actions
INFO: Running command line: bazel-bin/license.check.license_check ./formatted.txt -review -project automotive.score -repo https://github.com/eclipse-score/score -token otyhZ4eaRYK1tKLNNF-Y
[main] INFO Querying Eclipse Foundation for license data for 76 items.
[main] INFO Found 52 items.
[main] INFO Querying ClearlyDefined for license data for 24 items.
[main] INFO Found 24 items.
[main] INFO Vetted license information was found for all content. No further investigation is required.

@aschemmel-tech aschemmel-tech force-pushed the aschemmel-tech-config-mgt-init branch 2 times, most recently from 491603b to c421afa Compare April 10, 2025 07:10
masc2023
masc2023 requested changes Apr 10, 2025
View reviewed changes
Copy link
Contributor

@masc2023 masc2023 left a comment

Choose a reason for hiding this comment

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

Initial version ok, found one link, which should be changed after merge with main

docs/platform_management_plan/config_management.rst Outdated
Git defines branches as a means of parallel development. In the S-CORE project the following types of branches will be used:

* local branches - created from "remote" branches, in these the development of the contributors takes place, no restriction on naming.
* main branch - a "remote" branch which contains all the latest file versions checked by CI, reviewed, accepted and merged.
Copy link
Contributor

@masc2023 masc2023 Apr 10, 2025

Choose a reason for hiding this comment

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

may add restriction on naming, link, as stated for local no restriction?

Copy link
Contributor Author

@aschemmel-tech aschemmel-tech Apr 10, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

the main branch is always called "main", added

Copy link
Contributor

@masc2023 masc2023 Apr 10, 2025

Choose a reason for hiding this comment

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

ok, missunderstood, but then we hace naming conventions for local branches, see Contribute, General, Naming Conventions

masc2023
masc2023 previously approved these changes Apr 10, 2025
View reviewed changes
@aschemmel-tech aschemmel-tech force-pushed the aschemmel-tech-config-mgt-init branch from c421afa to bb5908f Compare April 10, 2025 11:35
@aschemmel-tech aschemmel-tech merged commit 5006e77 into main Apr 10, 2025
10 checks passed
@aschemmel-tech aschemmel-tech deleted the aschemmel-tech-config-mgt-init branch April 10, 2025 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@masc2023 masc2023 masc2023 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@aschemmel-tech @masc2023