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

Add bibat to packages.yml and Teddy Groves to contributors.yml #216

Merged
merged 3 commits into from
Jul 12, 2023

Conversation

teddygroves
Copy link
Contributor

I manually filled in as many fields as I could, but there were a few I couldn't easily do.

Hopefully what I added is ok, otherwise please let me know how to fill in the rest!

I didn't include all the gh_meta fields - hope it still works!
@teddygroves
Copy link
Contributor Author

pre-commit.ci autofix

@teddygroves
Copy link
Contributor Author

Here is the error message from https://github.com/pyOpenSci/pyopensci.github.io/actions/runs/5486952577/jobs/9997727510?pr=216 :

Running 3 checks (Images, Links, Scripts) in ["_site"] on *.html files ...


Checking 672 external links
Checking 48 internal links
Checking internal link hashes in 23 files
For the Links > External check, the following failures were found:

* At _site/about-peer-review/index.html:439:

  External link https://www.twitter.com/arianemsasso failed: got a time out (response code 0) (status code 0)

* At _site/blog/pyOpenSci-hiring-community-manager-spring-2023.html:449:

  External link https://geolatinas.org/ failed with something very wrong.
It's possible libcurl couldn't connect to the server, or perhaps the request timed out.
Sometimes, making too many requests at once also breaks things. (status code 0)

* At _site/our-community/index.html:9617:

  External link https://www.twitter.com/arianemsasso failed: got a time out (response code 0) (status code 0)


HTML-Proofer found 3 failures!
Ran on 44 files!

I think these errors don't come from this PR?

@lwasser
Copy link
Member

lwasser commented Jul 12, 2023

thank you for this @teddygroves i'll make sure this gets merged today!! welcome to our ecosystem!

those are spurious link failures - you are correct - totally unrelated to this PR! thank you again for being a part of our community!!

@lwasser lwasser merged commit 970f32b into pyOpenSci:main Jul 12, 2023
@lwasser
Copy link
Member

lwasser commented Jul 12, 2023

@all-contributors please add @teddygroves for code, review

@allcontributors
Copy link
Contributor

@lwasser

I've put up a pull request to add @teddygroves! 🎉

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