Skip to content

MAINT: adds plugin install tab for apple silicon arch #20

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

Merged
merged 5 commits into from
Sep 4, 2025
Merged

MAINT: adds plugin install tab for apple silicon arch #20

Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
4712b76
MAINT: adds plugin install tab for apple silicon arch
lizgehret Jul 10, 2025
0d6f4e1
fix newline spacing
lizgehret Jul 10, 2025
22796ed
Update src/lib/components/InstallPlugin.svelte
lizgehret Jul 10, 2025
6db3b12
shortened tab names
lizgehret Jul 10, 2025
c217f6e
reorder tabs
gregcaporaso Sep 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 14 additions & 2 deletions src/lib/components/InstallPlugin.svelte
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,22 @@
}

const create = $derived(`conda env create --name ${env_name} --file ${env_url}`);
const create_silicon = $derived(
[
"CONDA_SUBDIR=osx-64 conda env create",
`--name ${env_name}`,
`--file ${env_url}`,
`conda activate ${env_name}`,
"conda config --env --set subdir osx-64"
].join('\n')
);
const update = $derived(`conda env update --file ${env_url}`);

const ast = $derived(u('div', key({class: '-my-5'}), [u('tabSet', key(), [
u('tabItem', key({title: '[Fresh Install]'}), [
u('tabItem', key({title: '[Fresh Install (Apple Silicon)]'}), [
u('code', key({lang: 'bash', class: '!my-3'}), create_silicon),
]),
u('tabItem', key({title: '[Fresh Install (all other architectures)]'}), [
Copy link
Member

Choose a reason for hiding this comment

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

The titles end up being pretty long, I wonder if we can just have an exception for (Apple Silicon)?

Copy link
Member Author

Choose a reason for hiding this comment

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

yeah, i know... i do worry about it being confusing if we only have one arch specified. let me think about other name options to see if there's still a way to convey this that's less verbose

Copy link
Member Author

Choose a reason for hiding this comment

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

Screen Shot 2025-07-10 at 4 23 04 PM

How do you feel about this @ebolyen?

u('code', key({lang: 'bash', class: '!my-3'}), create),
]),
u('tabItem', key({title: '[Update Existing]'}), [
Expand All @@ -23,4 +35,4 @@

</script>

<MySTMinimal {ast}/>
<MySTMinimal {ast}/>