Skip to content

Minor textual review of text files in the repo.#220

Merged
barbosa merged 2 commits intoscanapi:masterfrom
supakeen:review-text
Jul 25, 2020
Merged

Minor textual review of text files in the repo.#220
barbosa merged 2 commits intoscanapi:masterfrom
supakeen:review-text

Conversation

@supakeen
Copy link
Copy Markdown
Contributor

README.md

  • English grammar and readability fixes.

CONTRIBUTING.md

  • Change versions link to point to a public link.
  • English grammar and readability fixes.

Copy link
Copy Markdown
Member

@camilamaia camilamaia 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 very much for the corrections! I left just minor comments

Comment thread CHANGELOG.md Outdated

## [Unreleased]

## [1.0.6] - 2020-07-25
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
## [1.0.6] - 2020-07-25

We keep all the changes under Unreleased section and only when we want in fact to create a new release, we move them to the proper release title, like this https://github.com/scanapi/scanapi/pull/212/files#diff-4ac32a78649ca5bdd8e0ba38b7006a1eR10

So for now, we can leave everything under ##[Unreleased]

Comment thread pyproject.toml Outdated
[tool.poetry]
name = "scanapi"
version = "1.0.5"
version = "1.0.6"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
version = "1.0.6"
version = "1.0.5"

@supakeen
Copy link
Copy Markdown
Contributor Author

I have updated the CHANGELOG format I misunderstood across my PR's :)

supakeen added 2 commits July 25, 2020 16:44
README.md
- English grammar and readability fixes.

CONTRIBUTING.md
- Change `versions` link to point to a public link.
- English grammar and readability fixes.
@barbosa barbosa merged commit 1a6607a into scanapi:master Jul 25, 2020
@camilamaia camilamaia mentioned this pull request Aug 25, 2020
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