Skip to content

update README with windows instructions#170

Merged
emduc merged 3 commits intomainfrom
readme_update
Sep 20, 2022
Merged

update README with windows instructions#170
emduc merged 3 commits intomainfrom
readme_update

Conversation

@emduc
Copy link
Contributor

@emduc emduc commented Sep 20, 2022

Small update to the read me to fix a few typos and add indications to launch the system without tmus.
Was way more straight forward than expected 🙈

@emduc emduc requested a review from nkcr September 20, 2022 14:44
@coveralls
Copy link

coveralls commented Sep 20, 2022

Pull Request Test Coverage Report for Build 3091068561

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 63.81%

Totals Coverage Status
Change from base Build 3053251873: 0.0%
Covered Lines: 3172
Relevant Lines: 4971

💛 - Coveralls

Copy link
Contributor

@nkcr nkcr left a comment

Choose a reason for hiding this comment

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

Great, thank you. 👍
Just added some minor formatting comments.

README.md Outdated
./setup.sh
```
With this other script you can choose the number of nodes that you want to set up:
With this other script using tmux you can choose the number of nodes that you want to set up:
Copy link
Contributor

Choose a reason for hiding this comment

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

Be sure to format the lines at 80 chars (In VS code the rewrap plugin can be of a great help):

With this other script using tmux you can choose the number of nodes that you
want to set up:

README.md Outdated

1: Run 3 nodes:
If you are using Windows and cannot use tmux, you need to do the actions of the
scripts in point _1_ and _2_ manually: open 3 terminal sessions and run the
Copy link
Contributor

Choose a reason for hiding this comment

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

We tend to use *text* for italic, instead of _text_

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@emduc emduc merged commit f101f1a into main Sep 20, 2022
@emduc emduc deleted the readme_update branch September 20, 2022 15: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