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

point testing badge testing page #208

Merged
merged 1 commit into from
Mar 17, 2024
Merged

Conversation

jGaboardi
Copy link
Member

Currently the CI badge on README links directly to the badge image. Updating that to link back to the action file page.

@knaaptime
Copy link
Member

i think theres also codecov stuff we need to do here ?

@jGaboardi
Copy link
Member Author

i think theres also codecov stuff we need to do here ?

We can either do in baby steps or I can add to this PR. Your call. (but my vote is baby steps).

@knaaptime
Copy link
Member

dont care for single or multi-step, but i was hoping you'd do it since you already figure out the segregation stuff :P

@jGaboardi
Copy link
Member Author

I will do it. Let's merge this then I will take another pass.

@knaaptime knaaptime merged commit 33ecefb into pysal:main Mar 17, 2024
0 of 6 checks passed
@jGaboardi jGaboardi deleted the readme_test_badge branch March 17, 2024 01:42
@jGaboardi
Copy link
Member Author

i think theres also codecov stuff we need to do here ?

@knaaptime What exactly do we need for codecov? Can you make a list in a new issue for me to look into?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants