Skip to content

Update dev container#2157

Merged
SimonYansenZhao merged 5 commits intostagingfrom
simonz/devcontainer
Aug 27, 2024
Merged

Update dev container#2157
SimonYansenZhao merged 5 commits intostagingfrom
simonz/devcontainer

Conversation

@SimonYansenZhao
Copy link
Collaborator

@SimonYansenZhao SimonYansenZhao commented Aug 27, 2024

Description

Dev containers and GitHub Codespaces provide more convenient development environments for Recommenders users. But the dev container configuration in the repo is not working and contains many deprecated settings. This PR modifies the dev container configuration including the following updates:

  • OS: Debian 12 $\to$ Ubuntu 24.04
  • Python: 3.7 $\to$ 3.10
  • Java: OpenJDK 8 $\to$ OpenJDK 21
  • Use Conda for env management
  • Install recommended VS Code extensions for Python
  • Add several settings for Python and Pytest
  • Specify minimum machine specification

Related Issues

References

Checklist:

  • I have followed the contribution guidelines and code style for this project.
  • I have added tests covering my contributions.
  • I have updated the documentation accordingly.
  • I have signed the commits, e.g. git commit -s -m "your commit message".
  • This PR is being made to staging branch AND NOT TO main branch.

Signed-off-by: Simon Zhao <simonyansenzhao@gmail.com>
Signed-off-by: Simon Zhao <simonyansenzhao@gmail.com>
Signed-off-by: Simon Zhao <simonyansenzhao@gmail.com>
Signed-off-by: Simon Zhao <simonyansenzhao@gmail.com>
Signed-off-by: Simon Zhao <simonyansenzhao@gmail.com>
Copy link
Collaborator

@miguelgfierro miguelgfierro left a comment

Choose a reason for hiding this comment

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

Great!

@SimonYansenZhao SimonYansenZhao merged commit d0a4af3 into staging Aug 27, 2024
@miguelgfierro miguelgfierro deleted the simonz/devcontainer branch August 27, 2024 10:45
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.

2 participants