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: force create joss cross ref badge #524

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

lwasser
Copy link
Member

@lwasser lwasser commented Nov 21, 2024

let's see if this addresses #517

{% if raw_string contains "github.com" %}
<!-- If github is in the link, it's a link to a tag -->
<li>
<a href="{{ raw_string }}" rel="permalink">
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This parses the string to determine what type or url it is. We could do this in pyosmeta to make life easier potentially. If it's a latest DOI, maybe we just use those words vs the archive. it's still a valid citation.

this also will fail if the DOI provided is off in some way (i haven't figured out why some are off but i'm assuming there could be typos etc).

{% endif %}
{% if apackage.astropy %}
True
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the line that currently isn't working for some reason. It could be that we need a better flag or it could be that it has to parse the list of communities associated with the package (which is likely the case).

@lwasser
Copy link
Member Author

lwasser commented Nov 21, 2024

points of failure now make sense - one is a bad website url. the second is a bad day (that I can see when we parse our packages but haven't debugged to understand the why.

Both of these items could be fixed via pyos meta if we do url validation. I set up URL validation for personal websites (grabbed from people's GitHub profiles). it would be easy also to check docs and dois as we parse them. This means we'd have to do less work on the website at the end of things.

@lwasser
Copy link
Member Author

lwasser commented Nov 21, 2024

this is what the page looks like now

https://output.circle-artifacts.com/output/job/486db103-bf6d-4fd0-9ecf-2eec13f09160/artifacts/0/_site/python-packages.html

we could merge this as is if people agree as it patches a handful of bugs. Then fix pyosmeta in december when I have a bit more help to handle the bigger workflow issues that this patches over with conditional statements :) !!

@pllim @willingc let me know what you think.

Copy link
Contributor

@pllim pllim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think there is one more:

[400] [https://doi.org/true](https://doi.org/true) | Failed: Network error: Bad Request

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