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

Upload a page for project #276

Merged
merged 4 commits into from
May 18, 2024
Merged

Conversation

dylanscottlow
Copy link
Contributor

Closes # .

Note this is a co-authored project.

Authors: Dylan Scott Low Y.J. and Stephanie Fu Pei-Yun

Copy link
Collaborator

@pbellec pbellec left a comment

Choose a reason for hiding this comment

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

Awesome project!

There's only a small issue in the header for the project to render properly.

I've suggested the changes (removing two lines).

Please accept the changes and I'll merge the project.

dylanscottlow and others added 2 commits June 13, 2023 12:33
Co-authored-by: Pierre Bellec <bellecp@iro.umontreal.ca>
Co-authored-by: Pierre Bellec <bellecp@iro.umontreal.ca>
@dylanscottlow
Copy link
Contributor Author

Hi Prof. Bellec,

I've committed the changes you requested. Please let me know if there is anything else needed!

@pbellec
Copy link
Collaborator

pbellec commented May 18, 2024

OMG so sorry I missed merging this :( LGTM merging now.

@pbellec pbellec merged commit 1db2366 into school-brainhack:main May 18, 2024
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.

None yet

2 participants