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

Add-badges-to-README.md #8

Merged
merged 11 commits into from
Oct 29, 2024
Merged

Add-badges-to-README.md #8

merged 11 commits into from
Oct 29, 2024

Conversation

sarg3nt
Copy link
Owner

@sarg3nt sarg3nt commented Oct 28, 2024

No description provided.

@sarg3nt sarg3nt assigned sarg3nt and BrickCForce and unassigned sarg3nt Oct 28, 2024
@sarg3nt sarg3nt requested a review from BrickCForce October 28, 2024 23:30
@sarg3nt
Copy link
Owner Author

sarg3nt commented Oct 29, 2024

@BrickCForce
Ignore the build error with trivy. We are having issues with running out of ghcr pulls. There's another PR that fixes it but for now don't let that stop you from approving.

1. Copy the `.mise.toml` file from the root of this repo to your project's root and modify it as needed.
1. The provided `.devcontainer` will automatically call `mise install` to install the custom versions of the applications.
1. After the container is started and you exec into it, `mise` will automatically install the app versions listed in the `.mise.toml` file.
1. Other applications can also be installed by editing the `.mise.toml` file.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Numbered steps were not incremented here.

Copy link
Owner Author

Choose a reason for hiding this comment

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

They are not supposed to be. This is Markdown best practice. The downstrem instance automatically numbers them correctly. This makes adding new items in random places and moving items around the list easier.

README.md Outdated
3. Copy the `.devcontainer` directory to your project.
4. Copy the following files to the root of your project.
These are optional but encouraged.
- `.mise.toml`: The config file for manageing specific versions of tooling you need for your Go project.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Minor type for 'managing'.

Copy link
Owner Author

Choose a reason for hiding this comment

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

Fixed

README.md Outdated
- [Dev Container Setup](#dev-container-setup)
- [`dev.sh`](#devsh)
- [Starship](#starship)
- [Initial Worksation Setup](#initial-worksation-setup)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Typo for 'workstation'? There seems to be 8 instances in total in this file.

Copy link
Owner Author

Choose a reason for hiding this comment

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

Fixed all

Copy link
Collaborator

@Stanrofl Stanrofl left a comment

Choose a reason for hiding this comment

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

Please fix spelling errors.

Add instructions as to where the dev container is stored and how to use it.
@sarg3nt sarg3nt merged commit e7debc0 into main Oct 29, 2024
1 of 2 checks passed
@sarg3nt sarg3nt deleted the Add-badges-to-README.md branch October 29, 2024 19:52
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