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

IDE: Support Codespaces #179

Merged
merged 2 commits into from
Jan 23, 2025
Merged

Conversation

AlexanderLanin
Copy link
Member

As described in the linked issue, this switches py_binary and py_library to aspect_rules_py. This way there is no dependency to the system python executable.

closes #178

As described in the linked issue, this switches py_binary and py_library to aspect_rules_py. This way there is no dependency to the system python executable.
@AlexanderLanin
Copy link
Member Author

AlexanderLanin commented Jan 17, 2025

Bug: bazel run //:format.fix still relies on system python.

vscode ➜ /workspaces/score (codespaces) $ bazel run //:format.fix
INFO: Analyzed target //:format.fix (1 packages loaded, 2017 targets configured).
INFO: Found 1 target...
Target //tools/format:format.fix up-to-date:
bazel-bin/tools/format/format.fix.bash
INFO: Elapsed time: 0.417s, Critical Path: 0.01s
INFO: 1 process: 1 internal.
INFO: Build completed successfully, 1 total action
INFO: Running command line: bazel-bin/tools/format/format.fix.bash
/usr/bin/env: ‘python3’: No such file or directory

'format.fix' requires a system python installation
@MaximilianSoerenPollak
Copy link
Contributor

Makes sense to have no setup/installation needed for anything, that seems to be the reason to use bazel.

LGTM 👍

@AlexanderLanin
Copy link
Member Author

Yeah they are basically competing technologies. But here I'm using the devcontainer just for installing bazel and bazel does the rest.

@AlexanderLanin AlexanderLanin added the community:infrastructure General Score infrastructure topics label Jan 23, 2025
@antonkri antonkri self-requested a review January 23, 2025 13:13
@AlexanderLanin AlexanderLanin merged commit eae387b into eclipse-score:main Jan 23, 2025
6 checks passed
@AlexanderLanin AlexanderLanin deleted the codespaces branch January 23, 2025 13:16
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.

Support Codespaces
3 participants