-
Notifications
You must be signed in to change notification settings - Fork 4
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
Added some info chla_introduction.md #35
Conversation
This pull request is being automatically built with GitHub Actions and Netlify. To see the status of your deployment, click below. 🔍 Git commit SHA: 61d3c43 |
@callumrollo It's me, hi! Trying to put some work into this as much as I can (and know haha) |
Hi Chiara, great that you're getting stuck in! The error in the build is indeed from the config. Looks like an issue in the environment, not caused by your PR so feel free to keep making similar contributions. I'll work on fixing the sphinx issue in a separate PR :) |
Lovely! Thanks! |
closing and reopening PR to trigger the CI and see if PR #36 fixed it |
You can now see a preview of the site in the "deployment preview url" from the bot: https://66bcae0020e3ad97e21e9c97--chla-sop.netlify.app/ Looks like the formatting isn't quite what we'd want, I think you want the 1) 2) 3) on separate lines? But now when you push a new commit it should update that deployment page and you can check how it looks |
This still isn't rendering quite right :/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good! Just a couple more things
Hi!
Trying to give some momentum this work. I have limited experience so happy to chat with people and figure out the best way forward =)