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

Fix spelling on "Recent contributors" Permalink on Navbar #543

Merged
merged 3 commits into from
Mar 31, 2022

Conversation

adeola-dev
Copy link
Contributor

@adeola-dev adeola-dev commented Mar 30, 2022

Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!

  • tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR -- or run tests locally with rake test
  • code is in uniquely-named feature branch and has no merge conflicts
  • PR is descriptively titled
  • PR body includes fixes #0000-style reference to original issue #
  • ask @publiclab/reviewers for help, in a comment below

We're happy to help you get this ready -- don't be afraid to ask for help, and don't be discouraged if your tests fail at first!

If tests do fail, click on the red X to learn why by reading the logs.

Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software

We have a loose schedule of reviewing and pulling in changes every Tuesday and Friday, and publishing changes on Fridays.

Thanks!

@welcome
Copy link

welcome bot commented Mar 30, 2022

Thanks for opening this pull request! Dangerbot will test out your code and reply in a bit with some pointers and requests.
There may be some errors, but don't worry! We're here to help! 👍🎉😄

@jywarren
Copy link
Member

Aha sorry - we need to fix this unrelated git protocol error, i'll see what i can do!

npm ERR! Error while executing:
npm ERR! /usr/bin/git ls-remote -h -t git://github.com/jywarren/github-api-simple.git
npm ERR! 
npm ERR! fatal: remote error: 
npm ERR!   The unauthenticated git protocol on port 9418 is no longer supported.
npm ERR! Please see https://github.blog/2021-09-01-improving-git-protocol-security-github/ for more information.
npm ERR! 
npm ERR! exited with error code: 128

@jywarren
Copy link
Member

Oops, i must've done that wrong... let me see...

npm ERR! Error while executing:
npm ERR! /usr/bin/git ls-remote -h -t ssh://[email protected]/jywarren/github-api-simple.git
npm ERR! 
npm ERR! Warning: Permanently added the RSA host key for IP address '140.82.112.3' to the list of known hosts.
npm ERR! [email protected]: Permission denied (publickey).
npm ERR! fatal: Could not read from remote repository.
npm ERR! 
npm ERR! Please make sure you have the correct access rights
npm ERR! and the repository exists.
npm ERR! 
npm ERR! exited with error code: 128

@adeola-dev
Copy link
Contributor Author

Hello @jywarren, what am I supposed to do next?

@jywarren
Copy link
Member

I'm sorry, i had to stop working on this for a bit. I just tried another fix. Fingers 🤞

Ref: publiclab/plots2#10841

Thank u for your patience!!

@jywarren jywarren merged commit 76cd5fc into publiclab:main Mar 31, 2022
@welcome
Copy link

welcome bot commented Mar 31, 2022

Congrats on merging your first pull request! 🙌🎉⚡️
Your code will likely be published to https://code.publiclab.org in the next few days.
In the meantime, can you tell us your Twitter handle so we can thank you properly?
Now that you've completed this, you can help someone else take their first step!
See: Public Lab's coding community!

@jywarren
Copy link
Member

Whew!!! Thank you!!! 🎉

@adeola-dev adeola-dev deleted the patch-1 branch April 1, 2022 15:49
@adeola-dev
Copy link
Contributor Author

Thank you for the assistance @jywarren! What are the next steps to take?

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