Skip to content

Changes to shared conda workflow#75

Open
yijunge-ucb wants to merge 1 commit intoberkeley-cdss:mainfrom
yijunge-ucb:main
Open

Changes to shared conda workflow#75
yijunge-ucb wants to merge 1 commit intoberkeley-cdss:mainfrom
yijunge-ucb:main

Conversation

@yijunge-ucb
Copy link
Copy Markdown
Collaborator

Changes to shared conda workflow

@yijunge-ucb
Copy link
Copy Markdown
Collaborator Author

@ryanlovett I made some changes to the docs. Before one instructor can have only one shared conda env, now they can have one for each course. I don't have permissions to merge anything in this repo. Let me know if you want me to make more changes.

@balajialg balajialg requested a review from ryanlovett April 1, 2026 20:11
Copy link
Copy Markdown
Contributor

@ryanlovett ryanlovett left a comment

Choose a reason for hiding this comment

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

I suggested some changes on one line.

conda create --prefix /srv/conda/envs/shared-<bcourse_id> python=3.11 -y
```

Make sure <bcourse_id> is identical to the bcourse_id that you provide to DataHub Admins. An instructor can request at maximum one shared conda environment for each course they teach.
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.

Suggested change:

Make sure <bcourse_id> is identical to the bCourses ID that you provide to DataHub Admins. An instructor can request a maximum of one shared conda environment for each course they teach.

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.

Note that I suggested backticks around <bcourse_id>, but it doesn't show up well in the comment formatting.

@ryanlovett
Copy link
Copy Markdown
Contributor

I was about to add you to the repo, but I think @balajialg must have gotten to it first.

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