review

[kaat0]

recommended:

• include other OSs in CI.yml for testing, for example:

      matrix:
        version:
          - 'lts' # latest LTS build
          - '1' # current
          - 'pre' # latest prerelease build (RCs, betas, and alphas
        julia-arch: [x64, x86, aarch64]
        os: [macOS-latest, ubuntu-latest, windows-latest]
        exclude:
          - os: macOS-latest
            julia-arch: x86
          - os: ubuntu-latest
            julia-arch: aarch64
          - os: windows-latest
            julia-arch: aarch64

• provide description for the different use of query() and turbo_url()

If helpful:

• Provide CRONTRIBUTION.md
• use all-contributors bot

[gwehrle]

Thanks a lot. I'll solve the recommended ones before release. As the other ones are not blocking a release I'll work on them afterwards.

[gwehrle]

@kaat0 Do you think the difference between query and turbo_url is clear now?

[kaat0]

Definitely.
What is your thought on automatically opening a browser with the provided query instead of printing it to the terminal? E.g. https://stackoverflow.com/questions/75475602/julia-how-to-open-a-url-into-browser

[gwehrle]

Would be pretty nice. I think I would prefer to have both options. With and without opening the browser. I tend to use a separate function like open_turbo_url(). The functionality is so different that a parameter feels a bit odd, but would also work.

I hope Desktop.jl works fine. It would be a little bit overpowered using LiveServer.jl for just this tiny feature.

[gwehrle]

@all-contributors please add @kaat0 for ideas and userTesting

[allcontributors]

@gwehrle

I've put up a pull request to add @kaat0! 🎉

[gwehrle]

Would be pretty nice. I think I would prefer to have both options. With and without opening the browser. I tend to use a separate function like open_turbo_url(). The functionality is so different that a parameter feels a bit odd, but would also work.

I hope Desktop.jl works fine. It would be a little bit overpowered using LiveServer.jl for just this tiny feature.

I'll transfer this discussion i a new Issue, so this can be closed.

[kaat0]

@all-contributors please add @gwehrle for ideas and userTesting and code

[allcontributors]

@kaat0

I've put up a pull request to add @gwehrle! 🎉