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

Fix documentation in Universum.Lifted.File to mention correct module #215

Merged
merged 1 commit into from
Aug 2, 2019

Conversation

serhiip
Copy link
Contributor

@serhiip serhiip commented Jul 22, 2019

Description

There is a typo in referenced module names - there is no file related functions in Data.Text (correct module is Data.Text.IO). This PR fixes the issue.

Issue could be observed here http://hackage.haskell.org/package/universum-1.5.0/docs/Universum-Lifted-File.html

✓ Checklist for your Pull Request

Ideally a PR has all of the checkmarks set.

If something in this list is irrelevant to your PR, you should still set this
checkmark indicating that you are sure it is dealt with (be that by irrelevance).

  • I made sure my PR addresses a single concern, or multiple concerns which
    are inextricably linked. Otherwise I should open multiple PR's.
  • If your PR fixes/relates to an open issue then the description should
    reference this issue. See also auto linking on
    github
    .

Related changes (conditional)

  • Record your changes

    • I added an entry to the changelog if my changes are visible to the users
      and

Stylistic guide (mandatory)

  • My commit history is clean (only contains changes relating to my
    issue/pull request and no reverted-my-earlier-commit changes) and commit
    messages start with identifiers of related issues in square brackets.

serhiip added a commit to serhiip/universum that referenced this pull request Jul 22, 2019
@serhiip serhiip marked this pull request as ready for review July 22, 2019 21:26
@gromakovsky
Copy link
Member

LGTM, thanks. We should merge #216 first to make CI pass here, then you'll have to rebase on latest master.

serhiip added a commit to serhiip/universum that referenced this pull request Jul 26, 2019
@serhiip
Copy link
Contributor Author

serhiip commented Jul 26, 2019

As #216 is merged I've rebased this one

CHANGES.md Outdated
@@ -6,12 +6,16 @@ Unreleased
from the list of changes suggested in `.hlint.yaml`.
* [#214](https:/serokell/universum/issues/214):
Update supported GHC versions (replace 7.10.3 with 8.6.5).

Copy link
Member

Choose a reason for hiding this comment

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

Looks like redundant space.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed

@gromakovsky gromakovsky requested a review from volhovm July 27, 2019 16:16
@gromakovsky
Copy link
Member

Can you please make history linear here?

serhiip added a commit to serhiip/universum that referenced this pull request Jul 27, 2019
serhiip added a commit to serhiip/universum that referenced this pull request Jul 27, 2019
@serhiip
Copy link
Contributor Author

serhiip commented Jul 27, 2019

Rebased everything onto master

@gromakovsky
Copy link
Member

Oh, and now the only remaining obstacle is this:

The base branch requires all commits to be signed.

@serhiip
Copy link
Contributor Author

serhiip commented Jul 30, 2019

Oh c'mon

@gromakovsky gromakovsky merged commit 039c117 into serokell:master Aug 2, 2019
@gromakovsky
Copy link
Member

Thanks for your patience.

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.

5 participants