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

README fixes #1570

Merged
merged 8 commits into from
Dec 29, 2022
Merged

README fixes #1570

merged 8 commits into from
Dec 29, 2022

Conversation

ilovehackathons
Copy link
Contributor

+nightly missing in build commands, grammar fixes, etc.

Have to choose between 'the' and 'our'.
Otherwise it fails with 'ERROR: cargo-contract cannot build using the "stable" channel.'.
Let's keep it consistent.
This is grammatically-correct.
Riot has been renamed to Element.
@codecov-commenter
Copy link

codecov-commenter commented Dec 29, 2022

Codecov Report

Merging #1570 (c213121) into master (45be514) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1570      +/-   ##
==========================================
- Coverage   71.61%   71.55%   -0.07%     
==========================================
  Files         205      205              
  Lines        6296     6296              
==========================================
- Hits         4509     4505       -4     
- Misses       1787     1791       +4     
Impacted Files Coverage Δ
crates/allocator/src/bump.rs 86.77% <0.00%> (-3.31%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@SkymanOne SkymanOne left a comment

Choose a reason for hiding this comment

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

Thank you!

@SkymanOne SkymanOne merged commit 83d5b43 into use-ink:master Dec 29, 2022
HCastano pushed a commit that referenced this pull request Jan 23, 2023
* Remove 'the' to correct the grammar

Have to choose between 'the' and 'our'.

* Remove the word 'file' to correct the grammar

It was repeated twice.

* Add '+nightly' to the build command

Otherwise it fails with 'ERROR: cargo-contract cannot build using the "stable" channel.'.

* Make the word 'examples' into a link

This is especially useful when viewing on https://paritytech.github.io/ink/.

* Add '+nightly' to another build command

They all need it.

* Add another missing '+nightly'

Let's keep it consistent.

* Use fields (plural) instead of field (singular)

This is grammatically-correct.

* Convert the Riot link into an Element link

Riot has been renamed to Element.
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.

None yet

3 participants