Skip to content

Improve development environment documentation #868

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

Merged
merged 2 commits into from
Apr 8, 2025

Conversation

AlexanderLanin
Copy link
Member

Enhance the clarity and completeness of the development environment setup instructions, including details on using a devcontainer and manual setup steps. Update formatting and add recommendations for IDE configurations.

Copy link

github-actions bot commented Apr 8, 2025

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
2025/04/08 06:43:52 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 (90 packages loaded, 10 targets configured)

Analyzing: target //:license-check (103 packages loaded, 16 targets configured)

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

Analyzing: target //:license-check (141 packages loaded, 1636 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)

Analyzing: target //:license-check (149 packages loaded, 4611 targets configured)

INFO: Analyzed target //:license-check (150 packages loaded, 4737 targets configured).
[11 / 13] [Prepa] JavaToolchainCompileBootClasspath external/rules_java~/toolchains/platformclasspath.jar
[12 / 13] Building license.check.license_check.jar (); 0s multiplex-worker
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: 18.260s, Critical Path: 2.42s
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 25 items.
[main] ERROR Error response from ClearlyDefined 429

Copy link

github-actions bot commented Apr 8, 2025

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

@AlexanderLanin AlexanderLanin requested a review from Copilot April 8, 2025 07:08
Copy link

@Copilot 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.

Copilot reviewed 1 out of 4 changed files in this pull request and generated no comments.

Files not reviewed (3)
  • .vscode/settings.json: Language not supported
  • docs/contribute/development/development_environment.rst: Language not supported
  • docs/contribute/development/ide/vscode.rst: Language not supported

@AlexanderLanin AlexanderLanin marked this pull request as ready for review April 8, 2025 07:10
Copy link
Contributor

@MaximilianSoerenPollak MaximilianSoerenPollak left a comment

Choose a reason for hiding this comment

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

Thanks for improving these.

@MaximilianSoerenPollak MaximilianSoerenPollak merged commit b35c741 into main Apr 8, 2025
9 of 10 checks passed
@MaximilianSoerenPollak MaximilianSoerenPollak deleted the alex-vscode-settings branch April 8, 2025 08:33
@AlexanderLanin AlexanderLanin added the community:infrastructure General Score infrastructure topics label Apr 11, 2025
@AlexanderLanin AlexanderLanin moved this from Todo to Done in Infrastructure Apr 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community:infrastructure General Score infrastructure topics
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants