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

2.0.0 ready to go? #1220

Closed
aearly opened this issue Jul 8, 2016 · 22 comments
Closed

2.0.0 ready to go? #1220

aearly opened this issue Jul 8, 2016 · 22 comments
Milestone

Comments

@aearly
Copy link
Collaborator

aearly commented Jul 8, 2016

I think its time. Everything in the milestone is complete, and the docs site is up and running. (except for the during inconsistency issue, which I think is just a docs fix)

I think 2.0.0 is ready for to be released into the wild!

cc @megawac

@aearly aearly added this to the 2.0 milestone Jul 8, 2016
@megawac
Copy link
Collaborator

megawac commented Jul 8, 2016

I agree after we resolve #1217

@gemox94
Copy link

gemox94 commented Jul 8, 2016

Hi. I'm starting with node so I was following a tuto where uses async.parallel. So when I wanted to run the app it throws me an error in async module and the error was that async couldn´t find applyEach module:
"Error: Cannot find module './applyEach'"
And I saw that the version of async was 2.0.0, so I decided to install an older version, specifically the version is 1.5.2 and the error disappeared.

@megawac
Copy link
Collaborator

megawac commented Jul 8, 2016

Ohhh, thats a good point @aearly we should test all the modules after we run the compile step. Can we configure the index.js file mocha uses to test?

@megawac
Copy link
Collaborator

megawac commented Jul 8, 2016

@gemox94 I actually can't reproduce this. Can you provide code to reproduce this issue?

@aearly
Copy link
Collaborator Author

aearly commented Jul 10, 2016

Well, last issue closed! I say we wait until Monday to release. It would be bad to inadvertently cause some issues when most people are off for the weekend.

@megawac
Copy link
Collaborator

megawac commented Jul 10, 2016

Sure, lets review the changelog and make sure its prepared

@tekwiz
Copy link
Contributor

tekwiz commented Jul 11, 2016

I just added an issue that I think should be resolved before v2 is released: #1225. I don't give a significant amount of weight to code-coverage results without a great deal of investigation into their configuration, but I feel this should be resolved before a major release.

That said, as a non-owner, I can say I am still 100% comfortable using the v2 release without code-coverage test results, so it may be more efficient to simply remove the Coveralls badge and circle back to resolve it afterwards.

@aearly
Copy link
Collaborator Author

aearly commented Jul 11, 2016

Coverage not working isn't a showstopper. We can fix this later. I don't know it it has worked at all since modularization.

@megawac
Copy link
Collaborator

megawac commented Jul 11, 2016

I think the issue is ES6 imports

@megawac
Copy link
Collaborator

megawac commented Jul 11, 2016

@aearly lets release tomorrow@noon (est), sound good?

@aearly
Copy link
Collaborator Author

aearly commented Jul 11, 2016

How about 1pm EST? (9am PST is a bit early)

@131
Copy link

131 commented Jul 12, 2016

It's time, it's time ! (furiously waiting for the -rc to disappear)

@megawac
Copy link
Collaborator

megawac commented Jul 12, 2016

Hehe, a couple issues got opened last night @131, we have to triage them before release. Waiting on @aearly feedback on a couple of the PRs before we go ahead

@megawac
Copy link
Collaborator

megawac commented Jul 12, 2016

https://github.com/caolan/async/milestone/1 for reference

@megawac
Copy link
Collaborator

megawac commented Jul 12, 2016

Some final things I noticed giving the jsdoc a final look over:

  1. document inserting multiple items into queue/cargo. Array form not documented.
  2. lib/iterator isn't used at all internally. Does this even deserve to live in async any longer? I'd say kill it in favour of ES-2015 iterators which natively support arrays.
  3. race to support iterables? (maybe v2.1?)
  4. waterfall to support iterables? (maybe v2.1?)

@aearly
Copy link
Collaborator Author

aearly commented Jul 12, 2016

  1. and 4) can wait for 2.1 . I also think Array literals are way more useful for those functions. Supporting generic iterables could lead to some interesting control flow options I haven't foreseen though. race could also support objects.

@aearly
Copy link
Collaborator Author

aearly commented Jul 12, 2016

I 👏 Think 👏 We're 👏 Ready 👏 To 👏 Go 😮

@megawac
Copy link
Collaborator

megawac commented Jul 13, 2016

@aearly
Copy link
Collaborator Author

aearly commented Jul 13, 2016

2.0.0 is live! https://www.npmjs.com/package/async

@aearly aearly closed this as completed Jul 13, 2016
@aearly
Copy link
Collaborator Author

aearly commented Jul 13, 2016

Now we wait for the inevitable bug reports... 😵 😅

@megawac
Copy link
Collaborator

megawac commented Jul 13, 2016

Do you have some unstaged changes to docs @aearly? Generating the docs on my machine don't turn out the same as gh-pages?

@aearly
Copy link
Collaborator Author

aearly commented Jul 13, 2016

Nope, everything is up to date on my end. What's different?

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

No branches or pull requests

5 participants