Just kicking the tires and making a text correction to the School Yearbook page

[kyttmacmanus]

Thank you for your contribution!
To help us understand and review your changes quickly, please complete the checklist and provide all relevant details.

Brief Summary

Corrects photos for Hieu Tran and Alber Sanchez

Type of Change

Please select the type of change your pull request introduces:

• [ X] Bug fix (fixes an issue)
• New feature (adds functionality)
• Documentation update (improves or adds documentation)
• Code style/refactor (improves the code structure, does not add functionality)
• Configuration change (modifies settings or configurations)
• Build/dependency update (changes build or dependencies)

[xames3]

@kyttmacmanus, you may need to raise another PR with a different branch. This seems like a merge request into the main branch. I'm unable to see the changed files, shows the whole commit backlog of 3 months.

[kyttmacmanus]

Thanks Akshay, I was a bit confused about where the merge should go. Just to make sure I understand, are you suggesting I create my own branch and then create PR to merge into Canary? Please let me know and thanks for the spoon-feeding.

…

On Mon, Jan 13, 2025 at 3:14 PM XA ***@***.***> wrote: @ kyttmacmanus, you may need to raise another PR with a different branch. This seems like a merge request into the main branch. I'm unable to see the changed files, shows the whole commit backlog of 3 months. — Reply to this email directly, ZjQcmQRYFpfptBannerStart This Message Is From an External Sender This message came from outside your organization. ZjQcmQRYFpfptBannerEnd @kyttmacmanus <https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_kyttmacmanus&d=DwMFaQ&c=009klHSCxuh5AI1vNQzSO0KGjl4nbi2Q0M1QLJX9BeE&r=N5AsY4jLonqxJEJI_QvE6za7XaSQVdPOqklIVrvVUGM&m=w4hbvvnGGufLjC2WkHo7IRZmw2EGFxF_1H51TOq0aRyj_U2Xdn1YgXxA0spVkXSq&s=HOTmYVqj9X-t7BfH6HRRv45W2MXcDPvvNQGoIzMrJgg&e=>, you may need to raise another PR with a different branch. This seems like a merge request into the main branch. I'm unable to see the changed files, shows the whole commit backlog of 3 months. — Reply to this email directly, view it on GitHub <https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_ciesin-2Dgeospatial_TOPSTSCHOOL-2DSPHINX_pull_60-23issuecomment-2D2588102044&d=DwMFaQ&c=009klHSCxuh5AI1vNQzSO0KGjl4nbi2Q0M1QLJX9BeE&r=N5AsY4jLonqxJEJI_QvE6za7XaSQVdPOqklIVrvVUGM&m=w4hbvvnGGufLjC2WkHo7IRZmw2EGFxF_1H51TOq0aRyj_U2Xdn1YgXxA0spVkXSq&s=zYtWtYZlvOi8nQTGgyTqnrAw-I2GwOvLpyJdP5i8MXc&e=>, or unsubscribe <https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ABCA5YBRQFIMQ6Q65QIJQ2T2KQNBXAVCNFSM6AAAAABVC7LH6CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKOBYGEYDEMBUGQ&d=DwMFaQ&c=009klHSCxuh5AI1vNQzSO0KGjl4nbi2Q0M1QLJX9BeE&r=N5AsY4jLonqxJEJI_QvE6za7XaSQVdPOqklIVrvVUGM&m=w4hbvvnGGufLjC2WkHo7IRZmw2EGFxF_1H51TOq0aRyj_U2Xdn1YgXxA0spVkXSq&s=Gvdmx4JNtrzEOiDFNdZcuJxo5dJc0mPEmbqM8BdjQrs&e=> . You are receiving this because you were mentioned.Message ID: ***@***.***>

-- Kytt MacManus (he/him) Applied Science Lead, Environmental Flow Physics Lab ( https://efpl.engineering.columbia.edu/) Columbia University School of Engineering and Applied Science Civil Engineering & Engineering Mechanics Department Adjunct Lecturer, E3B Dept of Columbia College

[xames3]

yes, you need to first fork the repository else git will you an error when you submit your changes. Once you fork the repository in your account, you can checkout your branch using command git checkout -b docs/kytt-latest. This will create your branch (docs/kytt-latest) and you can edit your changes in it and push it. Once that’s done, raise a PR to merge with canary and we can then merge it here. Hope that helps. I can help if you need any more information.

[kyttmacmanus]

Sorry for the delay, I just followed the described workflow and tried again. Please let me know if it looks good?

[xames3]

closing this one as it is merged in #61