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

[DOC] Adding a brief documentation for standalone sub-section's Steel book #160

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

kenaryn
Copy link

@kenaryn kenaryn commented Feb 7, 2024

This is a first iteration of the sub-section Steel book regarding to the standalone procedure in order to run Steel files from a terminal, either using cargo tool or independently.

This is my very first pull request in life so if something wrong or misplaced, please do alert me and I will try my best to amend it! :-)

5. Then run `exec elvish` and your are good to go! You can check your newly added path by typing:\
`pprint $paths`

- For those who still live in the dark caves of ancient shells like Bash or Zsh, simply feed the new path like the
Copy link
Contributor

@wmedrano wmedrano Feb 8, 2024

Choose a reason for hiding this comment

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

IMO: I would move bash and zsh up as they have higher adoption. Also remove language that makes any judgement on which is better to avoid alienating any users.

## Running a Steel file

6. Finally, compile and execute your Steel files from your terminal like so:\
`steel \<steel_file.scm\>`
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you add another section for how to install the built in modules?

The instructions right now are encoded in the CI: https://github.com/mattwparas/steel/blob/master/.github/workflows/rust.yml#L44

Copy link
Author

@kenaryn kenaryn Feb 8, 2024

Choose a reason for hiding this comment

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

I tried what I understood but some concepts are beyond me. What is a cog? Why adopting .steel as a default installation?
Do I had to add all install related commands?

Also, I clicked the Sync fork button in my personal repository then chose the Update branch option. Will it cause some inconveniences to target the upstream repository?
I am a very beginner so I am straightforward even if I could pass as a silly person doing so.

@kenaryn kenaryn requested a review from wmedrano February 8, 2024 20:26
Comment on lines 111 to 113
9. Finally, install the code coverage tool test, which will be of great use in the case you intend to contribute to
Steel development:
`cargo install cargo-tapaulin`
Copy link
Owner

Choose a reason for hiding this comment

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

I don't think this is necessary to include here

Comment on lines 99 to 100
You can try a hidden directory to avoid remove it by accident:
`export PATH=${PWD}/.steel:$PATH`
Copy link
Owner

@mattwparas mattwparas Feb 8, 2024

Choose a reason for hiding this comment

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

I don't have the .steel directory added to my path - I don't think it is a thing that needs to be endorsed here

@kenaryn kenaryn requested a review from mattwparas February 9, 2024 18:42
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