Skip to content

Change cows.list() function to return a promise (async/await)#44

Closed
jasonkhanlar wants to merge 1 commit intopiuccio:masterfrom
jasonkhanlar:asyncReaddir
Closed

Change cows.list() function to return a promise (async/await)#44
jasonkhanlar wants to merge 1 commit intopiuccio:masterfrom
jasonkhanlar:asyncReaddir

Conversation

@jasonkhanlar
Copy link
Contributor

No description provided.

@jasonkhanlar jasonkhanlar mentioned this pull request Nov 1, 2018
piuccio added a commit that referenced this pull request Nov 2, 2018
Allows users of the library to `await` for it

Closes #44
@piuccio piuccio mentioned this pull request Nov 2, 2018
@piuccio piuccio closed this in #45 Nov 2, 2018
piuccio added a commit that referenced this pull request Nov 2, 2018
Allows users of the library to `await` for it

Closes #44
@piuccio
Copy link
Owner

piuccio commented Nov 2, 2018

I merged another PR because your code would only work in node 10 (async / await). Promise as a constructor has been available since 0.12, so it's great for the minimum requirement of node 4

ofarukcaki pushed a commit to ofarukcaki/cowsay that referenced this pull request Jul 29, 2023
Allows users of the library to `await` for it

Closes piuccio#44
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