Skip to content

Remove optional_t from nimble#22

Open
def- wants to merge 1 commit into
flaviut:masterfrom
def-:master
Open

Remove optional_t from nimble#22
def- wants to merge 1 commit into
flaviut:masterfrom
def-:master

Conversation

@def-
Copy link
Copy Markdown
Contributor

@def- def- commented Nov 23, 2015

Doesn't look like it's required anymore.

@oprypin
Copy link
Copy Markdown
Collaborator

oprypin commented Nov 23, 2015

The thing is, nre itself is in the standard library, so there is not much point...

And I think this external library actually can't work anymore with its duplicate in the standard library, as the build fail shows.

@flaviut
Copy link
Copy Markdown
Owner

flaviut commented Nov 23, 2015

Yup. I'll keep this repo around for historical purposes, but you should
probably be using the version of nre in the standard library.

On Mon, Nov 23, 2015 at 10:11 AM Oleh Prypin notifications@github.com
wrote:

The thing is, nre itself is in the standard library, so there is not much
point...


Reply to this email directly or view it on GitHub
#22 (comment).

@flaviut flaviut force-pushed the master branch 6 times, most recently from 4489380 to a81131c Compare March 11, 2019 06:55
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