Skip to content

Conversation

@why-not-try-calmer
Copy link
Contributor

@why-not-try-calmer why-not-try-calmer commented Jan 6, 2022

Some new users might be put off by the lack of a array of choice between connection methods
that this package offers. This remark and the associated examples are a first attempt to make the package easier to start with.

@why-not-try-calmer why-not-try-calmer force-pushed the adrien/few-clarifications branch from 8027aeb to 2025b56 Compare January 6, 2022 22:47
@VictorDenisov VictorDenisov self-requested a review January 12, 2022 06:13
openReplicaSetSRV' :: HostName -> IO ReplicaSet
-- ^ Open /secure/ connections (on demand) to servers in a replica set. The seedlist and replica set name is fetched from the SRV and TXT DNS records for the given hostname. The value of 'globalConnectTimeout' at the time of this call is the timeout used for future member connect attempts. To use your own value call 'openReplicaSetSRV'''' instead.
-- The preferred connection method for cloud MongoDB providers.
-- ==== __Example__
Copy link
Member

Choose a reason for hiding this comment

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

Haddock requires special markers if you are giving a code example: https://haskell-haddock.readthedocs.io/en/latest/markup.html#code-blocks

You can also check what the formatted text looks like if you run

$ cabal haddock

It will tell you where it put generated docs.

@VictorDenisov
Copy link
Member

Could you also please follow commit message guidelines as described here: https://git-scm.com/book/en/v2/Distributed-Git-Contributing-to-a-Project

Some new users might be put off by the lack of a array of choice between connection methods
that this package offers. This remark and the associated examples are a first attempt to make the package easier to start with.
--
@why-not-try-calmer why-not-try-calmer force-pushed the adrien/few-clarifications branch from e7849d7 to 6ad28bc Compare January 15, 2022 15:24
@why-not-try-calmer why-not-try-calmer changed the title Some steps towards expliciteness Added a clarificatory remark and one example to openReplicaSet' Jan 15, 2022
@why-not-try-calmer
Copy link
Contributor Author

Any news from the front? :)

@VictorDenisov
Copy link
Member

VictorDenisov commented Feb 3, 2022

Sorry, I didn't realized that this new commit is different. Just tested it. Looks good. I'll merge this weekend.

@VictorDenisov VictorDenisov merged commit 5980bc1 into mongodb-haskell:master Feb 7, 2022
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.

2 participants