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

fix(artifacts): fix copypasta fail for run hint inference #450

Merged
merged 1 commit into from
Jun 26, 2023
Merged

fix(artifacts): fix copypasta fail for run hint inference #450

merged 1 commit into from
Jun 26, 2023

Conversation

msfjarvis
Copy link
Contributor

@msfjarvis msfjarvis commented Jun 26, 2023

The checks for PowerShell and sh were swapped around which resulted in generated sites recommending irm for shell scripts and curl for PowerShell ones :D

No tests failed post this change so I guess there is no coverage, I can add a test if someone can point to where it should go.

@msfjarvis
Copy link
Contributor Author

Rebased to pull in #451 for green checks

@shadows-withal
Copy link
Contributor

Weirdly, I'd expect https://opensource.axo.dev/oranda/ to be broken by this bug, but it seems to show the correct instructions:

image

@shadows-withal
Copy link
Contributor

@msfjarvis does this happen on any of your pages? just trying to get an overview of when exactly this breaks

@msfjarvis
Copy link
Contributor Author

@msfjarvis does this happen on any of your pages? just trying to get an overview of when exactly this breaks

It does. The repository is here, and the deployed page can be found here.

@shadows-withal
Copy link
Contributor

okay, weird, but it doesn't change our own site output when applying locally, so it's fine

@shadows-withal shadows-withal merged commit d71b425 into axodotdev:main Jun 26, 2023
7 checks passed
@msfjarvis msfjarvis deleted the hs/fix-script-inference branch June 26, 2023 10:13
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