Skip to content

[doc] Fix minor problems in README - #2463

Open
marnovandermaas wants to merge 1 commit into
lowRISC:masterfrom
marnovandermaas:minor_readme_fixes
Open

[doc] Fix minor problems in README#2463
marnovandermaas wants to merge 1 commit into
lowRISC:masterfrom
marnovandermaas:minor_readme_fixes

Conversation

@marnovandermaas

Copy link
Copy Markdown
Collaborator

No description provided.

Signed-off-by: Marno van der Maas <mvdmaas+git@lowrisc.org>
Comment thread README.md
Comment on lines 15 to +18
Ibex was initially developed as part of the [PULP platform](https://www.pulp-platform.org)
under the name ["Zero-riscy"](https://doi.org/10.1109/PATMOS.2017.8106976), and has been
contributed to [lowRISC](https://www.lowrisc.org) who maintains it and develops it further. It is
under active development.
contributed to [lowRISC](https://www.lowrisc.org) who maintains it and develops it further.
It is under active development.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
Ibex was initially developed as part of the [PULP platform](https://www.pulp-platform.org)
under the name ["Zero-riscy"](https://doi.org/10.1109/PATMOS.2017.8106976), and has been
contributed to [lowRISC](https://www.lowrisc.org) who maintains it and develops it further. It is
under active development.
contributed to [lowRISC](https://www.lowrisc.org) who maintains it and develops it further.
It is under active development.
Ibex was initially developed as part of the [PULP platform](https://www.pulp-platform.org) under the name ["Zero-riscy"](https://doi.org/10.1109/PATMOS.2017.8106976), and has been contributed to [lowRISC](https://www.lowrisc.org) who maintains it and develops it further.
It is under active development.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why not go for one sentence per line all the way through this doc?

@SamuelRiedel SamuelRiedel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I agree with Harry's comment. Otherwise, this looks good to me.

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