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 Infinite families of standard Cappell-Shaneson spheres #158

Merged
merged 1 commit into from
Oct 3, 2024

Conversation

AdityaK1729
Copy link
Contributor

fixes #155

@haraldschilly
Copy link
Member

Thank you! FYI, the "check" is here (linked in the box above).

@haraldschilly haraldschilly merged commit 5a8ee47 into sagemath:master Oct 3, 2024
1 check passed
@AdityaK1729
Copy link
Contributor Author

can you tell me more about checks, as i am new to this, like was I supposed to run those by ourself, if yes how can I do that in the future

@AdityaK1729 AdityaK1729 deleted the Add/iwaki2024infinite branch October 3, 2024 09:52
@haraldschilly
Copy link
Member

checks

The whole point is to avoid merging changes, which break the build process. So, e.g. if the syntax is wrong or whatever, the python script processing it will fail.

In this case here, the "check" runs that script, and the check passes if it does complete successfully.

As an external contributor, someone who owns the repository has to active running the checks – that's what I did. Then the check ran with your change. There was no error. Then I merged it.

If there would have been an error, you can see the error message. Then you're supposed to fix your code and push another commit. The check will run again.

The definition of what's going on in the check is here – in particular the last line: https://github.com/sagemath/publications/blob/master/.github/workflows/check.yml

@AdityaK1729
Copy link
Contributor Author

that helps thanks

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.

add "Infinite families of standard Cappell-Shaneson spheres"
2 participants