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

Topiary Book #750

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Topiary Book #750

wants to merge 1 commit into from

Conversation

ErinvanderVeen
Copy link
Collaborator

NAME OF PR

Issue: #

Description

Checklist

Checklist before merging:

  • CHANGELOG.md updated
  • README.md up-to-date

```nickel
{
languages = {
bash | priority 1 = {
Copy link
Member

Choose a reason for hiding this comment

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

I wonder if we should rather advise to use force. I think that's what we do in most of the Nickel documentation: if you want to override, force is the default choice, and only if you need something more complex then you can start thinking about numeric priorities.

@@ -0,0 +1,25 @@
# Using with Nix
Topiary provides a flake with several attributes. The main one is `topiary-cli`
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Topiary provides a flake with several attributes. The main one is `topiary-cli`
Topiary provides a flake with several outputs. The main one is `topiary-cli`

@Xophmeister Xophmeister mentioned this pull request Oct 1, 2024
2 tasks
@yannham
Copy link
Member

yannham commented Oct 17, 2024

@ErinvanderVeen I suppose you won't have much time to continue that. Should we try to take it over from now on? Or do you intend to make any progress in the foreseeable future?

Topiary's language query files (`.scm`) are located**. By default, you should
set it to `<local path of the topiary repository>/topiary-queries/queries`, for example:

```sh
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this meant to be a sh-session (interactive vs script)?


### nix install
#### On NixOS
```bash
Copy link
Contributor

Choose a reason for hiding this comment

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

These are definitely supposed to be sh-session (or shell-session, or console, depending on syntax highlighter)

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.

3 participants