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

Inactive PRs must be closed and their APIs added #2002

Open
matheusfelipeog opened this issue Aug 27, 2021 · 8 comments
Open

Inactive PRs must be closed and their APIs added #2002

matheusfelipeog opened this issue Aug 27, 2021 · 8 comments

Comments

@matheusfelipeog
Copy link
Collaborator

@matheusfelipeog matheusfelipeog commented Aug 27, 2021

There are many inactive PRs that have already been revised and with their APIs according to our addition criteria. These PRs are just waiting for some correction that was requested by one of the maintainers, and as they are inactive PRs, they will likely not be done.

Inactive PRs must be closed and their APIs added if they still meet our criteria.

Maintainers can add the APIs of these PRs in their respective sections directly on the master branch (since they have already been reviewed previously).

Contributors can open new PRs to add APIs of inactive PR.

Consider as inactive PR those that have no reply from their creator for more than 1 month.

To facilitate tracking, please mark the inactive PR in the new PR.

matheusfelipeog added a commit that referenced this issue Aug 27, 2021
Issue #2002 

resolve #1034
matheusfelipeog added a commit that referenced this issue Aug 27, 2021
issue #2002 

resolve #1106
@matheusfelipeog matheusfelipeog mentioned this issue Aug 27, 2021
8 tasks
@pawelborkar pawelborkar mentioned this issue Aug 29, 2021
8 tasks
@pawelborkar pawelborkar mentioned this issue Aug 29, 2021
8 tasks
@matheusfelipeog matheusfelipeog mentioned this issue Aug 31, 2021
8 tasks
matheusfelipeog pushed a commit that referenced this issue Sep 1, 2021
issue #2002 

resolve #1669
matheusfelipeog added a commit that referenced this issue Sep 1, 2021
issue #2002 

resolve #1168
matheusfelipeog pushed a commit that referenced this issue Sep 2, 2021
Issue #2002 
resolve #1916 

Squased:

* Added Geokeo geocoding API in alphabetical order

* Added Geokeo's Https support
@matheusfelipeog matheusfelipeog mentioned this issue Sep 2, 2021
8 tasks
matheusfelipeog added a commit that referenced this issue Sep 2, 2021
issue #2002 
resolve #1270
matheusfelipeog added a commit that referenced this issue Sep 2, 2021
issue #2002 
resolve #1295
@webdevrahul

This comment was marked as spam.

matheusfelipeog added a commit that referenced this issue Sep 14, 2021
issue #2002 
resolve #1388
@vamagithub
Copy link

@vamagithub vamagithub commented Sep 18, 2021

Can i work on this issue.

@priyanshm07
Copy link

@priyanshm07 priyanshm07 commented Oct 2, 2021

Are there any open issues?
@matheusfelipeog I would like to work on them.

@CipherGirl CipherGirl mentioned this issue Oct 2, 2021
8 tasks
@CipherGirl CipherGirl mentioned this issue Oct 2, 2021
8 tasks
@CipherGirl CipherGirl mentioned this issue Oct 2, 2021
8 tasks
@CipherGirl CipherGirl mentioned this issue Oct 2, 2021
8 tasks
@CipherGirl CipherGirl mentioned this issue Oct 2, 2021
8 tasks
@CipherGirl CipherGirl mentioned this issue Oct 2, 2021
8 tasks
@CipherGirl CipherGirl mentioned this issue Oct 2, 2021
8 tasks
@CipherGirl
Copy link
Contributor

@CipherGirl CipherGirl commented Oct 2, 2021

@matheusfelipeog Hey! I have raised some PRs. It would be great to have your feedback and your consideration about 'hacktoberfest-accepted' label :)

@iDeepverma iDeepverma mentioned this issue Oct 2, 2021
8 tasks
@megamayoy
Copy link

@megamayoy megamayoy commented Oct 2, 2021

@matheusfelipeog I wanna contribute during the Hacktoberfest. Can I work on this inactive PR #1678 ?

@matheusfelipeog
Copy link
Collaborator Author

@matheusfelipeog matheusfelipeog commented Oct 2, 2021

@matheusfelipeog Hey! I have raised some PRs. It would be great to have your feedback and your consideration about 'hacktoberfest-accepted' label :)

Excellent @CipherGirl, we will review all PRs as soon as possible. Note that there are many ;)

@matheusfelipeog
Copy link
Collaborator Author

@matheusfelipeog matheusfelipeog commented Oct 2, 2021

@matheusfelipeog I wanna contribute during the Hacktoberfest. Can I work on this inactive PR #1678 ?

Very good @megamayoy. Of course you can, you can create a new PR for any inactive PR as I described at the beginning of this issue:

Consider as inactive PR those that have no reply from their creator for more than 1 month.

matheusfelipeog added a commit that referenced this issue Oct 3, 2021
ref #2002 
resolve #1695
matheusfelipeog added a commit that referenced this issue Oct 3, 2021
ref #2002 
resolve #1690
@matheusfelipeog matheusfelipeog mentioned this issue Oct 3, 2021
8 tasks
matheusfelipeog added a commit that referenced this issue Oct 3, 2021
ref 1. #2002 
ref 2. #1500
matheusfelipeog added a commit that referenced this issue Oct 3, 2021
ref 1. #2002 
ref 2. #1500
matheusfelipeog added a commit that referenced this issue Oct 3, 2021
ref #2002 
resolve #1802
matheusfelipeog pushed a commit that referenced this issue Oct 3, 2021
ref: #2002 
resolve #1540
@matheusfelipeog matheusfelipeog mentioned this issue Oct 3, 2021
8 tasks
matheusfelipeog pushed a commit that referenced this issue Oct 3, 2021
ref #2002 
resolve #1481
micro-chipset added a commit to micro-chipset/public-apis that referenced this issue Oct 3, 2021
@OscarRG
Copy link

@OscarRG OscarRG commented Oct 3, 2021

Hey @matheusfelipeog i show that the Bible API is inactive and i created a PR to remove it from the list #2176. I added also a Harry Potter API to fetch the characters #2174. Let me know if there is any problem with the edits :)

@sleaper sleaper mentioned this issue Oct 3, 2021
8 tasks
@sleaper sleaper mentioned this issue Oct 3, 2021
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
7 participants